Skip to content

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 20, 2026

Previously compare_to was optional with a default of "base". Now it must be explicitly specified in all rule definitions, making the change detection baseline configuration explicit rather than implicit.

Changes:

  • Update schema to require compare_to field
  • Remove default value from rules parser
  • Update documentation to reflect required status
  • Add test for missing compare_to validation
  • Update all test rule definitions to include compare_to

claude and others added 3 commits January 18, 2026 22:20
Previously compare_to was optional with a default of "base". Now it must
be explicitly specified in all rule definitions, making the change
detection baseline configuration explicit rather than implicit.

Changes:
- Update schema to require compare_to field
- Remove default value from rules parser
- Update documentation to reflect required status
- Add test for missing compare_to validation
- Update all test rule definitions to include compare_to
All rule files now require the compare_to field. Updated:
- .deepwork/rules/architecture-documentation-accuracy.md
- .deepwork/rules/readme-accuracy.md
- .deepwork/rules/standard-jobs-source-of-truth.md
- .deepwork/rules/version-and-changelog-update.md
- src/deepwork/standard_jobs/deepwork_rules/rules/skill-md-validation.md

Also ran deepwork install to sync changes.
@nhorton nhorton merged commit d615a22 into main Jan 20, 2026
4 checks passed
@nhorton nhorton deleted the claude/require-compares-to-field-fwDHp branch January 20, 2026 18:25
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.

3 participants