FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(cli): package spec for CLI command by juliosuas · Pull Request #173 · google-labs-code/design.md · GitHub

fix(cli): package spec for CLI command - #173

Closed
juliosuas wants to merge 1 commit into
google-labs-code:mainfrom
juliosuas:fix/spec-command-package-path
Closed

fix(cli): package spec for CLI command#173
juliosuas wants to merge 1 commit into
google-labs-code:mainfrom
juliosuas:fix/spec-command-package-path

Conversation

juliosuas commented Aug 25, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes #153.

What changed

Copy docs/spec.md to both bundle entry-point directories during the CLI build. The linter entry point continues to resolve dist/linter/spec.md, while the CLI entry point can now resolve dist/spec.md as expected by getSpecContent().

Verification

This PR does not change .github/workflows. Org-level zizmor can still fail on pre-existing unpinned actions in test.yml; that is outside the scope of #153.

juliosuas force-pushed the fix/spec-command-package-path branch from d6212b3 to a214ee3 Compare August 25, 2026 16:15

Copy link
Copy Markdown
Contributor Author

Keeping this PR in scope of the #153 spec packaging fix (packages/cli/package.json only). I reverted the workflow/CI commits that hash-pinned actions/checkout, oven-sh/setup-bun, and actions/setup-node, added permissions: {}, and set persist-credentials: false.

Org-level zizmor (zizmor-output) fails on pre-existing unpinned actions in the repo's test.yml; that finding is not introduced by this diff. I am not expanding the PR to pin or rewrite CI.

juliosuas closed this by deleting the head repository Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Accidentally closed when the source fork was deleted during a profile cleanup. Same commits restored here: #175

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spec command fails on v0.3.0: reads dist/spec.md but the build emits dist/linter/spec.md

1 participant


Back | FazBrowse Home | New Git URL