Skip to content
This repository was archived by the owner on Oct 2, 2025. It is now read-only.

feat: Add public option to project config#757

Open
jmoseley wants to merge 5 commits intomainfrom
codex/add-public-option-to-project-configuration
Open

feat: Add public option to project config#757
jmoseley wants to merge 5 commits intomainfrom
codex/add-public-option-to-project-configuration

Conversation

@jmoseley
Copy link
Copy Markdown
Contributor

@jmoseley jmoseley commented Jun 10, 2025

Summary

  • allow project visibility via publicWorkflows option in config
  • parse project config with js-yaml
  • support --publicWorkflows option in new command
  • test project creation with --publicWorkflows flag

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
gensx-docs ⬜️ Skipped (Inspect) Jun 10, 2025 9:52pm
gensx-home ⬜️ Skipped (Inspect) Jun 10, 2025 9:52pm

@vercel vercel bot temporarily deployed to Preview – gensx-docs June 10, 2025 21:28 Inactive
@vercel vercel bot temporarily deployed to Preview – gensx-home June 10, 2025 21:28 Inactive
@jmoseley jmoseley force-pushed the codex/add-public-option-to-project-configuration branch from c3728d3 to 5cf4ae1 Compare June 10, 2025 21:42
@vercel vercel bot temporarily deployed to Preview – gensx-home June 10, 2025 21:42 Inactive
@vercel vercel bot temporarily deployed to Preview – gensx-docs June 10, 2025 21:42 Inactive
@vercel vercel bot temporarily deployed to Preview – gensx-home June 10, 2025 21:47 Inactive
@vercel vercel bot temporarily deployed to Preview – gensx-docs June 10, 2025 21:47 Inactive
@jmoseley jmoseley changed the title Add public option to project config feat: Add public option to project config Jun 10, 2025
@jmoseley jmoseley requested review from EvanBoyle, danfhernandez and dereklegenzoff and removed request for EvanBoyle and danfhernandez June 10, 2025 21:48
@vercel vercel bot temporarily deployed to Preview – gensx-home June 10, 2025 21:52 Inactive
@vercel vercel bot temporarily deployed to Preview – gensx-docs June 10, 2025 21:52 Inactive
projectName: z.string(),
environmentName: z.string().optional(),
description: z.string().optional(),
publicWorkflows: z.boolean().optional(),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd lean towards just calling this public. Spelling it out as publicWorkflows has better specificity but feels a bit clunky. Plus it would better align with the param in the CLI.

projectName: z.string(),
environmentName: z.string().optional(),
description: z.string().optional(),
public: z.boolean().optional(),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like there's a mismatch between here and the schema we use in the gensx package? public vs publicWorkflows

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants