Skip to content

README: list the handoff-needs-smoke-test hook - #535

Open
EdbertChan wants to merge 1 commit into
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527cfrom
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d
Open

README: list the handoff-needs-smoke-test hook#535
EdbertChan wants to merge 1 commit into
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527cfrom
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary

The main readme has a table listing every safety hook and what it does.

This adds the row for the new hook lower in this stack. That hook checks that a script handed to a person has been run first.

The PR checker counts the main readme as documentation, a separate kind of change, so the row lives in its own PR.

Review Claim

Approve one new row in the main readme's hook table, for the hook added at the bottom of this stack.

Review Lane

docs

Review Unit

docs

Safety Invariant

Only one table row is added. No code changes.

Slice Rationale

The row describes a hook added lower in this stack, so this lands after it. Keeping it separate lets the hook PR hold one kind of change.

Non-goals

  • Does not change the hook, its own guide, or its inventory row.
  • Does not list the login-shell hook; that hook is not in this stack.

Test Plan

Test Plan

Every CI command, run on this commit:

rc=0  bash scripts/run_all_tests.sh
rc=0  python3 scripts/check_no_tracked_local_artifacts.py
rc=0  python3 scripts/check_hook_test_coverage.py
rc=0  python3 scripts/check_rules_are_wired.py
rc=0  python3 scripts/check_skills_three_harnesses.py
rc=0  python3 scripts/check_ecosystem_boundaries.py
rc=0  python3 scripts/check_skill_file_refs.py
rc=0  python3 scripts/check_skill_test_coverage.py --base origin/main --head HEAD
rc=0  python3 scripts/check_skill_test_debt_no_growth.py
rc=0  python3 scripts/check_skill_trigger_mechanism.py
rc=0  python3 scripts/check_dora_baseline.py
rc=0  python3 scripts/check_no_dated_provenance.py --base origin/main
rc=0  python3 scripts/check_no_new_comments.py --base origin/main
rc=0  shellcheck install.sh

Revert Plan

Revert Plan
  • Safe to revert? Yes
  • Revert command: git revert <sha>
  • Post-revert steps: None. The row leaves the readme table.
  • Data migration? No

🤖 Generated with Claude Code

Depends-On: #511

@cursor

cursor Bot commented Sep 13, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_6980cd14-9a4a-4131-a5b6-715c8daa7ad5)

@EdbertChan

EdbertChan commented Sep 13, 2026

Copy link
Copy Markdown
Owner Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 hook: a script handed to the user is a claim that it runs #506
2 audit(token_audit): count "bogus" and "it didn't run" as the same class #507
3 gate: a branch taken per operating system needs a test that injects one #509
4 audit(token_audit): one submission carrying two slash commands is one message #510
5 audit: use the transcript's own markers for what the human actually sent #511
6 README: list the two new hooks from this stack #535 👈

@EdbertChan EdbertChan changed the title README: list the handoff-needs-smoke-test and remote-payload-collapses hooks README: list the two new hooks from this stack Sep 13, 2026
Adds the hook's row to the README hook table. The row moved out of the
hook PR because the PR checker counts the root README as its own docs
unit; the hook's own README and inventory row stay with the hook.

Change-Id: I0618d41d327180cc18b1b8daf4820706f912ffc1
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from 4894e27 to e1ac807 Compare September 13, 2026 18:39
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from 062b3db to 5c0e9dc Compare September 13, 2026 18:39
@EdbertChan EdbertChan changed the title README: list the two new hooks from this stack README: list the handoff-needs-smoke-test hook Sep 13, 2026
@cursor

cursor Bot commented Sep 13, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_e5f100c5-e9ef-4e81-9d56-f1151354611b)

@EdbertChan

Copy link
Copy Markdown
Owner Author

Revision history

# Type Changes Reason Date
1 initial 062b3db 2026-09-13 18:39 UTC
2 content 062b3db → 5c0e9dc (raw) 2026-09-13 18:39 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant