Run sync/parity workflows on OVVO_SYNC_TOKEN; defer autofix agent - #8
Merged
Conversation
…crets Use the repo's existing GitHub PAT secrets instead of a new PARITY_DISPATCH_TOKEN: OVVO_SYNC_TOKEN emits the inspect->autofix repository_dispatch, PARITY_APP_TOKEN opens the fix PR. Document that ANTHROPIC_API_KEY is a separate Anthropic credential and is still required.
Per current direction, hold the claude-code-action agent for later and keep the current method (maintainer applies reviewed parity fixes). - parity-autofix.yml: drop the claude-code-action step and ANTHROPIC_API_KEY / PARITY_APP_TOKEN dependency and the model input. On a live-R divergence (or when live R cannot be verified) it now opens a human parity-review PR carrying the reports, using OVVO_SYNC_TOKEN. - inspect-r-api-update.yml / sync-from-nns-core.yml: open their PRs with OVVO_SYNC_TOKEN so they trigger native-backend-ci; dispatch already uses it. - docs/parity_autofix.md: document the single-token model and how to enable the agent later (when ANTHROPIC_API_KEY is added). Automation only; no runtime/package behavior changes.
OVVO-Financial
changed the title
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
2 participants
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.
Two related changes:
1. Single-token model — everything runs on
OVVO_SYNC_TOKENPR #7 merged just before the token reconciliation, so
mainstill referenced anonexistent
PARITY_DISPATCH_TOKEN(auto-chain silently skipped). This pointsevery GitHub operation in the sync/parity workflows at the existing
OVVO_SYNC_TOKENsecret:inspect-r-api-update.yml—nns-parity-divergencedispatch and theinspection PR.
sync-from-nns-core.yml— the NNS-core sync PR.parity-autofix.yml— the parity-review PR.Using a real PAT (not the default
GITHUB_TOKEN) means these PRs also triggernative-backend-ci.2. Defer the autofix agent; keep the current method
The
anthropics/claude-code-actionstep is removed for now (to be re-addedlater).
parity-autofix.ymlstill installs live R at the recorded commit andre-runs the mapped parity, but on a divergence (or when live R can't be verified)
it now opens a human parity-review PR carrying the reports — a maintainer
applies the reviewed fix. Consequences:
ANTHROPIC_API_KEYneeded while the agent is deferred.PARITY_APP_TOKENis no longer referenced by any workflow.docs/parity_autofix.mdrewritten: single-token model + how to enable theagent later.
Automation only; no runtime/package behavior changes. All four workflow YAMLs
parse; no residual
ANTHROPIC_API_KEY/PARITY_DISPATCH_TOKEN/ agentreferences outside explanatory comments.
https://claude.ai/code/session_017KnASettwNxUdYJzchRLux