Skip to content

docs(spec-loop): sync specs with #1453 - #1454

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:sync-specs-20260928-2
Sep 28, 2026
Merged

potiuk merged 1 commit into
apache:mainfrom
potiuk:sync-specs-20260928-2

Conversation

@potiuk

@potiuk potiuk commented Sep 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Brings the specs up to date with fix(security): make the last sandbox-hostile gh calls plain, and fit the review block to PR imports #1453, the only commit merged since the last sync (Sync specs with contributed functionality #1452).
    tools/spec-loop/.last-sync now points at 77884f5c.
  • repo-health-family.md: workflow-security-audit no longer passes zizmor --gh-token "$(gh auth token)", because a gh inside $(…) stays sandboxed.
    zizmor reads its token from GH_TOKEN, GITHUB_TOKEN or ZIZMOR_GITHUB_TOKEN.
    With none set, the skill scans a shallow clone made by a plain gh repo clone, or asks the user to run the remote scan.
  • adversarial-review.md: the shared pre-PR review block now also covers skills that verify or import someone else's PR (security-issue-import-from-pr).
    Those skills run the reviewers before reporting on or acting on that PR.
  • security-issue-lifecycle.md: security-issue-invalidate Step 1 now runs gh issue view as a plain command and reads the JSON it prints, instead of redirecting it into a file.

Type of change

  • Other: spec-loop specs and sync marker

Test plan

  • prek run --all-files passes
  • Each change was checked against the code on main at 77884f5c.

Generative AI tooling was used to author this PR: Claude Code (Claude Opus 5).

🤖 Generated with Claude Code

Back-fill the specs for the one commit merged since the last sync:
workflow-security-audit takes zizmor's token from the environment
instead of a sandboxed `$(gh auth token)`, security-issue-invalidate runs
its last redirected `gh` call as a plain command, and the pre-PR
adversarial-review block also covers skills that verify or import
someone else's PR. Advance .last-sync to the current main.

Generated-by: Claude Opus 5
@potiuk
potiuk merged commit 48c046b into apache:main Sep 28, 2026
9 checks passed
@potiuk potiuk added the family:tools tools/* label Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant