Repository navigation
Optional codebase-context integration for work-plan and work-execute - #130
PatrickSys wants to merge 2 commits into
Conversation
Add off-by-default integrations.codebaseContext config, helper commands that record plan context and edit preflight receipts under .work/, workflow docs, and unit tests with a stubbed CLI runner. Co-authored-by: Patrick Rosselló Colom <PatrickSys@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2fe3ab0075
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
|
|
||
| if (first === 'codebase-context') { | ||
| return cmdCodebaseContextSubcommand(second, [...rest, ...(json ? ['--json'] : [])], { |
There was a problem hiding this comment.
For the enabled integration with an available CLI, both documented workflow commands lose their required inputs here: rest comes from positionalArgs(), which removes --files, --plan, --query, and --task together with their values. Consequently, plan-context returns missing_files and edit-preflight returns missing_query, even when callers supply all required flags, so neither workflow can produce its context or preflight artifact. Forward the original workspace-resolved subcommand arguments rather than the filtered positional list.
Useful? React with 👍 / 👎.
Document Workspine-only plan-context/edit-preflight helpers, invoke patterns and search --intent explore|edit (MCP get_team_patterns / search_codebase), bundle integration modules in .work/bin, forward CLI flags correctly, and run node for .js command paths. Tests mirror real JSON shapes; optional CC_E2E_BIN e2e passes against main build. Co-authored-by: Patrick Rosselló Colom <PatrickSys@users.noreply.github.com>
Adds an off-by-default
integrations.codebaseContextbridge to the codebase-context CLI.Naming:
plan-contextandedit-preflightare Workspine helper subcommands only (not codebase-context CLI commands). They call the real CLI:patterns+ per-filesearch --intent explorefor planning, andsearch --intent editfor edit preflight (MCPget_team_patterns/search_codebase).When enabled and the CLI is available, artifacts live under
.work/integrations/codebase-context/. Disabled or missing CLI paths returndisabled/unavailablewith no workflow change.Includes helper wiring in
.work/bin/gsdd.mjs, User Guide notes, stubbed unit tests aligned to real JSON shapes, and an optionalCC_E2E_BINe2e test against a built codebase-context from main.