fix(submodule): sync PMOVES-Archon to upstream/main + track main branch - #1594
Conversation
Add kilo-brief-check and shift-crew-trail PostToolUse hooks with settings.json matchers. Wire Phase 6 TAC subtree (n4090.shift-crew) covering 7 skill nodes and 2 hooks for the 4090 node. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- POWERFULMOVES/PMOVES-Archon:main force-synced to coleam00/Archon:main (v0.3.12, f4d68296) - .gitmodules: archon submodule branch updated from PMOVES.AI-Edition-Hardened to main — tracks the stable upstream release branch - Submodule pointer bumped to f4d68296 Note: PMOVES.AI-Edition-Hardened remains on POWERFULMOVES/PMOVES-Archon for PMOVES customization work. Rebase onto new main is a separate task. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Review limit reached
Your plan currently allows 1 review/hour. Refill in 13 minutes and 44 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![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: fbc431fd41
ℹ️ 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".
| PAYLOAD="$($PYTHON_CMD -c "import json,sys; print(json.dumps({'event':'shift_crew_edit','file':sys.argv[1],'branch':sys.argv[2],'node':'4090-claude','ts':sys.argv[3]}))" "$FILE_BASENAME" "$BRANCH" "$TIMESTAMP" 2>/dev/null)" || PAYLOAD="{}" | ||
| # Build payload | ||
| TIMESTAMP="$(date -u +%Y-%m-%dT%H:%M:%SZ 2>/dev/null || python3 -c 'from datetime import datetime, timezone; print(datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ"))')" | ||
| PAYLOAD="{\"event\":\"shift_crew_edit\",\"file\":\"$(basename "$FILE_PATH")\",\"branch\":\"$BRANCH\",\"node\":\"4090-claude\",\"ts\":\"$TIMESTAMP\"}" |
There was a problem hiding this comment.
Constructing PAYLOAD via shell string interpolation drops JSON escaping for dynamic fields, so a valid Git branch name containing " (for example feat/quote"name, which git check-ref-format --branch accepts) produces malformed JSON and breaks downstream parsing of this hook event. The previous implementation used Python json.dumps to serialize safely; this line reintroduces an input-dependent payload corruption path.
Useful? React with 👍 / 👎.
#1595) PMOVES-Archon:PMOVES.AI-Edition-Hardened is now cleanly rebased onto upstream/main (v0.3.12 / f4d68296) with PMOVES-specific additions: - CODEOWNERS, Dependabot, CodeQL configuration - Claude Code MCP adapter for PMOVES.AI integration - CHIT secrets manifest, env tier loading, health/announcer/registry modules - docker-compose.pmoves.yml, PMOVES.AI_INTEGRATION.md context - CI workflow triggers for PMOVES.AI-Edition-Hardened branch Previously (#1594) the tracking branch was set to main (pure upstream) to unblock the upstream sync. Now that hardened is properly based on main, restore the integration submodule to track the PMOVES-hardened branch so the deployed Archon service includes PMOVES integration modules. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 participant
Summary
POWERFULMOVES/PMOVES-Archon:mainforce-synced tocoleam00/Archon:main(v0.3.12,f4d68296).gitmodules: archon submodulebranchupdated fromPMOVES.AI-Edition-Hardened→mainf4d68296Root cause:
PMOVES.AI-Edition-Hardenedwas trackingcoleam00/Archon:devand was 130 commits behind.PMOVES-Archon:mainitself was also unsynced from upstream. Both now point to the stablecoleam00/Archon:mainrelease.PMOVES.AI-Edition-Hardened remains on
POWERFULMOVES/PMOVES-Archonwith the PMOVES customizations (GitHub App token minting, NATS auth, PBKDF2 hardening, persona service, deployment hooks). Rebasing those onto the new main base is a follow-on task.Test plan
git submodule status pmoves/integrations/archonshowsf4d68296POWERFULMOVES/PMOVES-Archon:mainmatchescoleam00/Archon:maintip🤖 Generated with Claude Code