From 537bc5057123be9386ef099fb8853a1b665e3031 Mon Sep 17 00:00:00 2001 From: app_modernization Date: Thu, 23 Apr 2026 19:42:09 +0800 Subject: [PATCH 1/3] docs(config): add updating/syncing instructions Explain how to pull upstream changes and re-sync framework files to ~/.claude/ using `node init.js --config --force`. Co-Authored-By: Claude Opus 4 --- docs/configuration.md | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/docs/configuration.md b/docs/configuration.md index 6d8a899..9249868 100644 --- a/docs/configuration.md +++ b/docs/configuration.md @@ -91,6 +91,24 @@ Location: `~/.claude/settings.json` Add project-specific permissions as needed (e.g., `Bash(mvn *)` for Java projects). +## Updating / Syncing + +Dev Pilot is actively maintained. When the upstream repo has new commands, agents, or features, sync them to your local `~/.claude/` with: + +```bash +cd dev-pilot && git pull +node init.js --config ~/.claude/pilot.yaml --force +``` + +| Flag | What it does | +|------|-------------| +| `--config ~/.claude/pilot.yaml` | Reuses your existing configuration instead of prompting for setup | +| `--force` | Overwrites framework files (commands, agents) with the latest versions | + +> **Note:** `--force` alone (without `--config`) will trigger the interactive setup wizard and create a new `pilot.yaml`, overwriting your existing one. Always pair `--force` with `--config` to preserve your configuration. + +Your `CLAUDE.md` is never overwritten by `init.js`. + ## Environment Variables The dashboard and lib modules also read these environment variables: From a7d4d51d65c4f537696c6987e8d6e422a7b6385f Mon Sep 17 00:00:00 2001 From: app_modernization Date: Thu, 23 Apr 2026 19:46:02 +0800 Subject: [PATCH 2/3] docs(config): add watch_pr settings to template and field reference Add watch_pr.auto_fix_ci and watch_pr.auto_fix_comments to pilot.yaml.template and docs/configuration.md field reference table. Co-Authored-By: Claude Opus 4 --- docs/configuration.md | 2 ++ pilot.yaml.template | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/docs/configuration.md b/docs/configuration.md index 9249868..2431d0f 100644 --- a/docs/configuration.md +++ b/docs/configuration.md @@ -46,6 +46,8 @@ defaults: | `build.test_command` | string | auto-detect | Override test command. `{{file}}` is replaced with the test file path | | `build.default_branch` | string | `main` | Default branch for git operations | | `test_runner_skill` | string | — | Name of a custom test runner skill in `~/.claude/skills/` (advanced — for complex test workflows) | +| `watch_pr.auto_fix_ci` | bool | `true` | Auto-fix CI failures detected by `/pilot-watch-pr` | +| `watch_pr.auto_fix_comments` | bool | `false` | Auto-fix review comments detected by `/pilot-watch-pr` | ### Preset Configs diff --git a/pilot.yaml.template b/pilot.yaml.template index 97b5a07..4ffdb5c 100644 --- a/pilot.yaml.template +++ b/pilot.yaml.template @@ -52,3 +52,8 @@ defaults: # (e.g., cross-repo builds, manual verification scenarios). # If omitted, /pilot-dev-issue uses inline build/test with auto-detection. # test_runner_skill: my-custom-test-runner + +# PR monitoring settings (used by /pilot-watch-pr) +# watch_pr: +# auto_fix_ci: true # default: true — auto-fix CI failures +# auto_fix_comments: false # default: false — auto-fix review comments From b481d95f905a5d0cd89e3a50df74861e2dcb3e0f Mon Sep 17 00:00:00 2001 From: app_modernization Date: Thu, 23 Apr 2026 19:58:11 +0800 Subject: [PATCH 3/3] fix(watch-pr): check unresolved comments for COMMENTED reviewDecision Previously only CHANGES_REQUESTED and REVIEW_REQUIRED triggered the GraphQL review thread query. PRs with reviewDecision COMMENTED were skipped, missing actionable review comments. Co-Authored-By: Claude Opus 4 --- docs/configuration.md | 2 -- framework/commands/claude/pilot-watch-pr.md | 2 +- framework/commands/copilot/pilot-watch-pr.md | 2 +- 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/docs/configuration.md b/docs/configuration.md index 2431d0f..1599b02 100644 --- a/docs/configuration.md +++ b/docs/configuration.md @@ -109,8 +109,6 @@ node init.js --config ~/.claude/pilot.yaml --force > **Note:** `--force` alone (without `--config`) will trigger the interactive setup wizard and create a new `pilot.yaml`, overwriting your existing one. Always pair `--force` with `--config` to preserve your configuration. -Your `CLAUDE.md` is never overwritten by `init.js`. - ## Environment Variables The dashboard and lib modules also read these environment variables: diff --git a/framework/commands/claude/pilot-watch-pr.md b/framework/commands/claude/pilot-watch-pr.md index 51c758c..6d992e9 100644 --- a/framework/commands/claude/pilot-watch-pr.md +++ b/framework/commands/claude/pilot-watch-pr.md @@ -90,7 +90,7 @@ az repos pr list \ #### Fetch unresolved review threads (GitHub only): -For each PR with `reviewDecision` of `CHANGES_REQUESTED` or `REVIEW_REQUIRED`: +For each PR with `reviewDecision` of `CHANGES_REQUESTED`, `REVIEW_REQUIRED`, or `COMMENTED`: ```bash gh api graphql -f query=' query($owner:String!,$repo:String!,$number:Int!) { diff --git a/framework/commands/copilot/pilot-watch-pr.md b/framework/commands/copilot/pilot-watch-pr.md index 30c594e..fc6efc8 100644 --- a/framework/commands/copilot/pilot-watch-pr.md +++ b/framework/commands/copilot/pilot-watch-pr.md @@ -84,7 +84,7 @@ az repos pr list \ #### Fetch unresolved review threads (GitHub only): -For each PR that has `reviewDecision` of `CHANGES_REQUESTED` or `REVIEW_REQUIRED`: +For each PR that has `reviewDecision` of `CHANGES_REQUESTED`, `REVIEW_REQUIRED`, or `COMMENTED`: ```bash gh api graphql -f query=' query($owner:String!,$repo:String!,$number:Int!) {