Remove stale GUI smoke note; add forensic-review skill for both agents - #106
Merged
Merged
Conversation
Its content has been overtaken by what actually shipped: PR #103 it called "proposed" merged three PRs ago, its cancellation-scenario count and Phase I sample are stale, and its deferred-decision advice against touching the reachability classifier's observation grouping is now contradicted by findings from reviewing PRs #103-#105. The EC-28 framing it introduced already lives in DEFECT-BACKLOG.md. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Both tools apparently use different config paths for project skills, so this keeps the skill under both .claude/ and .agents/ rather than guessing which one either tool actually reads. Excludes scheduled_tasks.lock in both directories, which is local runtime state, not repo content. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
1 participant
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
Documentation/tooling only — no source or test changes.
docs/GUI-SMOKE-FUTURE-NOTE.md: its content is superseded (PR Harden GUI smoke-test evidence #103 it called "proposed" merged three PRs ago, its cancellation-scenario count and Phase I sample are stale, and its advice against touching the reachability classifier's observation grouping is now contradicted by review findings from PRs Harden GUI smoke-test evidence #103-Distinguish unprocessed files from completed work after cancellation #105). The EC-28 framing it introduced already lives indocs/DEFECT-BACKLOG.md.encodingchecker-forensic-reviewskill under bothsources/.claude/skills/andsources/.agents/skills/, since Claude Code and Codex apparently read different config paths and it's unconfirmed which one Codex actually uses.scheduled_tasks.lockis excluded from both — local runtime state, not repo content.🤖 Generated with Claude Code