docs(CLAUDE.md): agent operations notes from parallel-bot session exp… - #348
Merged
Conversation
…erience
Captures five operational findings from the parallel-claude
coordinator-feedback turn (2026-05-24), as a single "Agent
operations notes" section:
* CI signal reliability — "PR merged" does NOT mean "build green"
on this repo today; auto-merge fires through red builds.
Recently-merged PRs (#334/#335/#336/#344) all landed with `build`
red; the red persisted until PR #346.
* Reading CI logs — WebFetch on the Actions UI returns React
skeleton; use mcp__github__pull_request_read get_check_runs /
get_status, hand back to user for actual log lines via
`gh run view --log-failed`.
* Known-failing baseline checks — vscode-smoke (npm 404),
migration-assistant (fixed by #342 but stale-base branches red),
governance/Language anti-pattern policy (flags approved TS
exemptions), Hypatia 143-finding comment (mostly the same
exemption hits). Don't waste turns investigating per-PR; only
investigate *changes* in this set.
* Branching discipline — `git fetch origin main && git rebase
origin/main` immediately before push, not just at branch-
creation. Claude 1's #337 accidentally reverted #334+#335 from
a stale base; cheap to prevent.
* Test-fixture hygiene — when adding a new declaration shape
(extern fn, etc.), test it against EVERY downstream consumer
(parse/resolve/typecheck/interp/codegen). PR #346's FnExtern
interp bug survived since the interpreter was written because
no test fed an inline `extern fn` to Interp.eval_program.
Pure documentation. Zero behavioural risk.
Refs PR #346, #335, #337, #344
🔍 Hypatia Security ScanFindings: 130 issues detected
View findings[
{
"reason": "Stray AI.a2ml in root -- use 0-AI-MANIFEST.a2ml only",
"type": "banned",
"file": "AI.a2ml",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Superseded by 0-AI-MANIFEST.a2ml",
"type": "banned",
"file": "AI.djot",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Issue in quality.yml",
"type": "missing_workflow",
"file": "quality.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Issue in security-policy.yml",
"type": "missing_workflow",
"file": "security-policy.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action hyperpolymath/standards/.github/workflows/governance-reusable.yml@main needs attention",
"type": "unpinned_action",
"file": "governance.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Action actions/checkout@v6 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action denoland/setup-deno@v2 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/example/smoke_driver.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/cli.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
Sixth operational finding, captured this session: when a PR is
squash-merged the squashed commit on main has a new SHA, distinct
from the source-branch commits. If you reset your local branch to
main, `git status` reports N "ahead of origin/branch" — but those
N are just main-side commits the obsolete remote-tip never saw,
not unpushed work.
Adds:
* Recognition checklist (PR closed/merged, local matches main,
remote tip is pre-merge SHA, `git log origin/<branch>..HEAD`
shows other people's work).
* Two safe fixes by intent:
git push origin --delete <branch> # done
git push --force-with-lease origin <branch> # align
* Note: --force-with-lease is safe here because there's no
unmerged remote work to clobber.
Pure documentation. Same branch / same PR shape as the previous
ops-notes commit.
Refs PR #344 (the trigger case)
🔍 Hypatia Security ScanFindings: 130 issues detected
View findings[
{
"reason": "Stray AI.a2ml in root -- use 0-AI-MANIFEST.a2ml only",
"type": "banned",
"file": "AI.a2ml",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Superseded by 0-AI-MANIFEST.a2ml",
"type": "banned",
"file": "AI.djot",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Issue in quality.yml",
"type": "missing_workflow",
"file": "quality.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Issue in security-policy.yml",
"type": "missing_workflow",
"file": "security-policy.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action hyperpolymath/standards/.github/workflows/governance-reusable.yml@main needs attention",
"type": "unpinned_action",
"file": "governance.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Action actions/checkout@v6 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action denoland/setup-deno@v2 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/example/smoke_driver.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/cli.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
🔍 Hypatia Security ScanFindings: 130 issues detected
View findings[
{
"reason": "Stray AI.a2ml in root -- use 0-AI-MANIFEST.a2ml only",
"type": "banned",
"file": "AI.a2ml",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Superseded by 0-AI-MANIFEST.a2ml",
"type": "banned",
"file": "AI.djot",
"action": "delete",
"rule_module": "root_hygiene",
"severity": "high"
},
{
"reason": "Issue in quality.yml",
"type": "missing_workflow",
"file": "quality.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Issue in security-policy.yml",
"type": "missing_workflow",
"file": "security-policy.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action hyperpolymath/standards/.github/workflows/governance-reusable.yml@main needs attention",
"type": "unpinned_action",
"file": "governance.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Action actions/checkout@v6 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action denoland/setup-deno@v2 needs attention",
"type": "unpinned_action",
"file": "publish-jsr.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/example/smoke_driver.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/cli.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/affinescript/affinescript/affinescript-deno-test/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
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.
…erience
Captures five operational findings from the parallel-claude coordinator-feedback turn (2026-05-24), as a single "Agent operations notes" section:
CI signal reliability — "PR merged" does NOT mean "build green" on this repo today; auto-merge fires through red builds. Recently-merged PRs (feat(stdlib): STDLIB-04a — Mut effect externs (make_ref/get/set) → real impl (Closes #328) #334/feat(borrow): NLL last-use expiry for ref-bindings (CORE-01 pt3 Slice A, Refs #177) #335/docs(adr): CORE-02 / #234 / ADR-016 — truth ledger to "DELIVERED" #336/Claude/happy mendel b4 ie u #344) all landed with
buildred; the red persisted until PR fix(interp):eval_declhandlesFnExtern(#328 build-failure root cause) #346.Reading CI logs — WebFetch on the Actions UI returns React skeleton; use mcp__github__pull_request_read get_check_runs / get_status, hand back to user for actual log lines via
gh run view --log-failed.Known-failing baseline checks — vscode-smoke (npm 404), migration-assistant (fixed by ci(migration-assistant): fix smoke-parse for tree-sitter-cli 0.25 #342 but stale-base branches red), governance/Language anti-pattern policy (flags approved TS exemptions), Hypatia 143-finding comment (mostly the same exemption hits). Don't waste turns investigating per-PR; only investigate changes in this set.
Branching discipline —
git fetch origin main && git rebase origin/mainimmediately before push, not just at branch- creation. Claude 1's chore(stdlib): STDLIB-04c — remove deadstring_concatextern (Closes #330) #337 accidentally reverted feat(stdlib): STDLIB-04a — Mut effect externs (make_ref/get/set) → real impl (Closes #328) #334+feat(borrow): NLL last-use expiry for ref-bindings (CORE-01 pt3 Slice A, Refs #177) #335 from a stale base; cheap to prevent.Test-fixture hygiene — when adding a new declaration shape (extern fn, etc.), test it against EVERY downstream consumer (parse/resolve/typecheck/interp/codegen). PR fix(interp):
eval_declhandlesFnExtern(#328 build-failure root cause) #346's FnExtern interp bug survived since the interpreter was written because no test fed an inlineextern fnto Interp.eval_program.Pure documentation. Zero behavioural risk.
Refs PR #346, #335, #337, #344