chore(spec-loop): sync specs to main and advance .last-sync - #1313
Merged
Merged
Conversation
Reconciles the spec-loop specs with the functionality that shipped between 0e7012b and 5c75abd: - agent-isolation-sandbox: records the write-deny over the vetted-ops **policy** on both surfaces it resolves to — the plugin-cache install and the committed `.apache-magpie-overrides/tools/vetted-ops/**` override — why it applies even though the write dispatcher is never excluded from the sandbox, and why the framework's own dispatcher source at `tools/vetted-ops/` is deliberately left un-denied. apache#1311 landed the excludedCommands list and the "excluded code must not be agent-writable" rule that this builds on. - sandbox-diagnostics: an eighth troubleshooting entry (the touch overlay's runtime-state directory denied) and the branch-order rule the hint hook follows where two entries both end in "agent refused operation". - security-reporting: the per-issue event cache is trusted only when it was written after that issue's `updatedAt`, so a relabelled issue is refetched instead of being served stale label history. Generated-by: Claude Opus 5
potiuk
force-pushed
the
sync-specs-20260921-045009
branch
from
September 21, 2026 07:38
be2267f to
4ae2353
Compare
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
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
tools/spec-loop/.last-syncfrom0e7012bato87be68b8(the currentmaintip) and brings the three specs that had fallen behind level with what shipped.agent-isolation-sandbox— documents the two non-ghsandbox.excludedCommandsentries (the vetted-ops read dispatcher, and the optional eval runner), the rule that anything excluded must execute code the agent cannot write, and the independent write-deny over the vetted-ops policy on both surfaces it resolves to — including thesandbox_write_deniedprek entry the in-repo override needs (fix(sandbox): deny agent edits to the vetted-ops policy #1308, fix(ci): exclude the vetted-ops policy from the sandbox-write-denied hooks #1309, fix(vetted-ops): exclude the read dispatcher from the sandbox #1310), and whytools/vetted-ops/itself is deliberately left un-denied.sandbox-diagnostics— seven troubleshooting entries becomes eight (the touch overlay's runtime-state directory denied, fix(agent-isolation): keep the touch overlay's state out of /tmp #1306), plus the branch-order rule the hint hook follows where two entries both end inagent refused operation.security-reporting— the per-issue event cache is trusted only when it was written after that issue'supdatedAt, so a relabelled issue is refetched instead of being served stale label history.No functional change: specs only.
Type of change
.claude/skills/<name>/) — eval fixtures updated belowtools/<system>/*.md)tools/*/withpyproject.toml)docs/,README.md,CONTRIBUTING.md)projects/_template/)prek, workflows, validators)tools/spec-loop/spec sync +.last-syncbumpTest plan
prek run --all-filespasses (includingspec-validateand the workspacepytest)uv run pytest/ruff check/mypypasses.last-syncpointerRFC-AI-0004 compliance
No principle is touched: the change adds no mutation, no host access, no
LLM hop. It records sandbox and privacy behaviour that already shipped.
Linked issues
Refs #1306, #1308, #1309, #1310.
Notes for reviewers (optional)
Worth spot-checking that the
agent-isolation-sandboxwording matcheswhat the merged settings actually do — in particular the distinction
between the read dispatcher (excluded) and
vetted-opitself (neverexcluded, because
--calleris argv), and the claim that the in-repopolicy override needs a
sandbox_write_deniedentry orprek's fixersabort with
Operation not permitted.🤖 Generated with Claude Code