Adopt per-run subdirectory output pattern for validate-audit#2219
Merged
Trecek merged 8 commits intoMay 8, 2026
Merged
Conversation
…ills
Change validate-audit (project-local), validate-test-audit, and the extended
validate-audit SKILL.md files from writing flat timestamped files into a
shared validate-audit/ directory to creating per-run timestamped subdirectories
(validate-audit-{YYYY-MM-DD_HHMMSS}/) with timestamp-free filenames inside.
Updates:
- Project-local .claude/skills/validate-audit/SKILL.md: critical constraints,
mkdir command, all file paths, Output Location section
- skills_extended/validate-test-audit/SKILL.md: same pattern updates
- skills_extended/validate-audit/SKILL.md: fallback path now creates per-run
dir; timestamp suffixes removed from all filenames
- skill_contracts.yaml: pattern_examples updated for both skills
- Tests: T-VAL-014, T-VTA-008 updated; T1/T2/T5/T6 new tests added
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
… symmetry The validated_report_path gate token was the only path reference without a human-readable label prefix in the headless output block. Summary/Manifest/ Tickets/Contested all had label: format; validated_report_path used key=value only. Restores the Report: line alongside the gate token. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…write instruction
The phrase "(relative to the current working directory)" directly contradicts the
{{AUTOSKILLIT_TEMP}} prefix which expands to an absolute path. Removes the
misleading parenthetical added in 41fd59c.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Both tests used prefix-only match {{AUTOSKILLIT_TEMP}}/validate-audit- which
would pass for paths like validate-audit-legacy/. T-VAL-014 now checks the full
validate-audit-{YYYY-MM-DD_HHMMSS}/ pattern. T-VTA-008 now checks the full
{{AUTOSKILLIT_TEMP}}/validate-audit-{YYYY-MM-DD_HHMMSS}/ pattern, consistent
with T-VTA-027 and matching the path structure in each SKILL.md.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
T-VAL-014 checks the BUNDLED_EXTENDED validate-audit which uses bash date
syntax ($(date ...)) not the {YYYY-MM-DD_HHMMSS} literal — prefix assertion
is correct for that skill. The CWD parenthetical in validate-test-audit is
required by test_file_producing_skills_have_cwd_anchor contract test and
cannot be removed without also updating that contract.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…grouping T-VAL-026 tests a general extended-validate-audit filename convention (no timestamp suffix) unrelated to review_decisions. Moving it from TestValidateAuditReviewDecisions to TestValidateAuditContent where general content assertions live. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ock and remove slop parenthetical
- Sync extended SKILL.md headless block with project-local: add `Report: {validated_report_path}` label before the machine-readable sentinel lines (C1: Report: was added to project-local by bd31ae2 but not the extended skill)
- Remove `(a newly created per-run directory)` parenthetical from Output Location section (F1: colon already signals list follows; implied by mkdir)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Trecek
deleted the
validate-audit-adopt-per-run-subdirectory-output-pattern/1960
branch
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
Change validate-audit (project-local), validate-test-audit, and the extended validate-audit SKILL.md files from writing flat timestamped files into a shared
validate-audit/directory to creating a per-run timestamped subdirectory (validate-audit-{YYYY-MM-DD_HHMMSS}/) with timestamp-free filenames inside. This matches the established pattern used byvalidate-team. Update downstream path references inskill_contracts.yamland all affected tests.Closes #1960
Implementation Plan
Plan file:
/home/talon/projects/autoskillit-runs/impl-20260507-225440-809232/.autoskillit/temp/make-plan/validate-audit_adopt_per-run_subdirectory_output_pattern_plan_2026-05-07_225500.md🤖 Generated with Claude Code via AutoSkillit
Token Usage Summary
* Step used a non-Anthropic provider; caching behavior may differ.
Token Efficiency
Model Usage Breakdown