Conversation
tmchow
changed the title
doctor diagnostic skill and extend setup with dependency management
tmchow
force-pushed
the
feat/setup-dependency-management
branch
from
deb3038 to
df389f4
Compare
tmchow
force-pushed
the
feat/setup-dependency-management
branch
3 times, most recently
from
36be991 to
bcea41b
Compare
![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: bcea41b82e
ℹ️ 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".
tmchow
changed the title
doctor diagnostic skill and extend setup with dependency management
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13f0fb7a60
ℹ️ 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".
…management Add a new /doctor skill that checks environment health for the Compound Engineering plugin — CLI tools, MCP servers, env vars, plugin version, and .gitignore. Uses a bundled check script for single-pass execution with emoji status output. Extend /setup with a Phase 1 dependency flow that invokes /doctor for diagnostics, then offers guided installation of missing tools with per-dependency env var prompting. Includes centralized dependency registry, requirements doc, and implementation plan.
…design Replace the older setup and todo planning chain with the consolidated March 25 requirements and plan, and update setup/doctor to match the new repo-local config cleanup model. This also removes stale Context7 dependency references that had been reintroduced on the branch after rebasing onto main.
Rename the setup and doctor skills to ce-setup and ce-doctor, and update the active docs and references to match the new command names and file paths.
…e-and-fix flow ce-doctor was a read-only diagnostic that always led to "now run /ce-setup to fix." Eliminating the extra step: ce-setup now runs diagnostics directly (Phase 1) then offers remediation (Phase 2). Key changes: - Delete ce-doctor skill entirely - Move check-health script into ce-setup/scripts/ with config-at-top refactor (deps declared as array, single loop for check + output) - Remove dependency-registry.md (script is now the single source of truth for dependency metadata) - Remove ffmpeg, MCP server checks, and env var checks from scope - Fix agent-browser URL (nichochar -> vercel-labs) and gitignore check (grep -Fqx -> git check-ignore) - Project section only appears when issues exist - Script no longer tells user to "run /ce-setup" (circular since it runs inside ce-setup) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ce-setup can now create .compound-engineering/config.local.yaml from a bundled template when the file doesn't exist yet. Template ships in references/config-template.yaml with all settings commented out. The step also ensures the file is gitignored. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace internal terminology ("legacy config", "Codex delegation",
"review agent selection") with plain developer-friendly wording.
Consolidate /ce-setup and /ce-update reminders into a single line.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ce-update is Claude Code-only. Pre-resolve CLAUDE_PLUGIN_ROOT at skill load time and conditionally include the /ce-update tip so Codex/Gemini users don't see a dead reference. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…encies Replace rtk with artifact-recording tools (vhs, silicon, ffmpeg) that support creating GIFs, code screenshots, and feature demo videos for PRs and sharing.
tmchow
force-pushed
the
feat/setup-dependency-management
branch
from
13f0fb7 to
b62b5c8
Compare
…rsing - Redirect git rev-parse stdout to /dev/null so in_repo captures only "yes"/"no" instead of "true\nyes" - Guard --version flag against missing argument to prevent infinite loop - Resolve project check paths from git toplevel instead of CWD - Update SKILL.md Step 5 to instruct repo-root-relative config creation Addresses PR #345 review feedback. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ca62a30694
ℹ️ 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".
Cleaner completion message with a visible finish marker, and aligned comments with a section header in the config template.
…alls Setup now copies config.local.example.yaml into the project repo (committed, always refreshed) alongside the gitignored config.local.yaml. The health check detects missing or outdated example files. Install commands use NONINTERACTIVE=1 for brew and CI=true for npm to prevent prompts during agent-driven installs. Gitignore uses a *.local.yaml wildcard for future-proofing.
- Add welcome message before health check runs - Promote vhs/silicon/ffmpeg from optional to recommended - Use quiet non-interactive brew installs (HOMEBREW_NO_AUTO_UPDATE=1, -q) - Show labeled tool list and config status in completion message - Remove redundant script header/dividers (agent text handles greeting) - Use warn (yellow) instead of fail (red) for missing deps
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a578e04db1
ℹ️ 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".
…ction - Step 4 now resolves repo root before deleting compound-engineering.local.md - Fix case pattern to match brew commands with env var prefixes Addresses PR #345 review feedback.
Merged
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
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Running
/ce-setupused to show a stub message. Now it diagnoses your environment, bootstraps config files, and installs missing tools in one interactive flow.Demo
What changed
Unified diagnose-and-fix skill. The old
/setupstub and a separate/ce-doctorconcept are now a single/ce-setupwith two phases: diagnose (run a bundled health-check script) then fix (config bootstrapping, dependency installs). One entry point.Health-check script.
scripts/check-healthruns a single-pass diagnostic covering CLI tool availability, project config state, and example config freshness. Adding a dependency is a one-line change to the script'sdepsarray. Missing tools show as yellow (recommended, not errors).Config bootstrapping. Setup creates two files in
.compound-engineering/:config.local.example.yaml(committed, always refreshed from the plugin template so teammates see available settings)config.local.yaml(gitignored via*.local.yamlwildcard, personal preferences)Non-interactive installs. All install commands suppress prompts for agent-driven runs:
NONINTERACTIVE=1 HOMEBREW_NO_AUTO_UPDATE=1 brew install -qfor Homebrew packages,CI=true npm install -g --no-audit --no-fund --loglevel=errorfor npm. agent-browser install also adds the skill vianpx skills add.Completion message. Shows a labeled tool list with green indicators and config status so results are visible without expanding collapsed script output.
Test plan
/ce-setupwith all tools installed: verify labeled tool list and "setup complete"/ce-setupwith missing tools: verify yellow indicators, multiSelect, install flowconfig.local.example.yamlgets refreshed on re-runconfig.local.yamlis gitignored and not overwritten on re-run