Skip to content

chore(tooling): an accepted warning raised to an error is not "resolved" (template#101) - #54

Merged
astrapi69 merged 1 commit into
mainfrom
chore/template-101
Oct 12, 2026
Merged

astrapi69 merged 1 commit into
mainfrom
chore/template-101

Conversation

@astrapi69

Copy link
Copy Markdown
Owner

Template wave for astrapi69/adaptive-learner-content-template#101 (merged as #104). Four template-owned files, byte-identical to the template's main: scripts/quality_state.py, scripts/validate_with_engine.mjs, tests/test_quality_state.py, .github/workflows/engine-validate.yml.

  • The runner prints the engine's rename table (RENAMED W-X -> E-X, from RENAMED_RULE_IDS and currentRuleId) with --warnings, and errors with their rule ID.
  • quality_state.py follows an accepted record to a rule the engine raised to an error and reports the raise instead of "resolved"; accept refuses an error.
  • The warnings summary step also runs after a failed gate.

Checked locally before the PR: pin 0.40.0, tests/test_quality_state.py 25 passed, runner exit 0 (80 warning(s)), and the warnings summary of the old and new quality_state.py on this repository's runner output is byte-identical. This repository has no accepted warnings, so its summary does not change.

🤖 Generated with Claude Code

…ed" (template#101)

Byte-identical copy of the template-owned files from
adaptive-learner-content-template#104: the engine runner prints the
engine's rename table and errors with their rule ID, quality_state.py
follows an accepted record to a raised rule, and the warnings summary
also runs after a failed gate.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@astrapi69
astrapi69 merged commit c7cc69d into main Oct 12, 2026
6 checks passed
@astrapi69
astrapi69 deleted the chore/template-101 branch October 12, 2026 16:13
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.

2 participants