DEVX-8: Fix classification gaps — expand DEFAULT_INFRASTRUCTURE #18

Merged
emil merged 0 commits from DEVX-8-fix-classification-gaps into master 2026-06-22 21:12:17 +00:00
Owner

Summary

Root cause analysis of GRM-64 misclassification revealed 28 files (scripts/**, REVIEW_CHECKLIST.md) were incorrectly classified as user-facing because common project file patterns were missing from DEFAULT_INFRASTRUCTURE.

Root Cause

DEFAULT_INFRASTRUCTURE was missing common project-level files that are not part of the installed package:

  • Project docs: CONTRIBUTING.md, CODE_OF_CONDUCT.md, REVIEW_CHECKLIST.md
  • Build tooling: uv.lock
  • Lint config: .checkmake.ini, .editorconfig
  • Git config: .gitattributes
  • Agent config: .devin/**

Fix

Added all missing patterns to DEFAULT_INFRASTRUCTURE. Added regression test that verifies all common project files are covered.

Impact

With this fix, the GRM-64 migration (28 changed scripts/** files + REVIEW_CHECKLIST.md) would be correctly classified as workflow-only — no unnecessary release triggered.

All 822 tests pass, 100% coverage.

Closes DEVX-8

## Summary Root cause analysis of GRM-64 misclassification revealed 28 files (scripts/**, REVIEW_CHECKLIST.md) were incorrectly classified as user-facing because common project file patterns were missing from DEFAULT_INFRASTRUCTURE. ### Root Cause DEFAULT_INFRASTRUCTURE was missing common project-level files that are not part of the installed package: - Project docs: CONTRIBUTING.md, CODE_OF_CONDUCT.md, REVIEW_CHECKLIST.md - Build tooling: uv.lock - Lint config: .checkmake.ini, .editorconfig - Git config: .gitattributes - Agent config: .devin/** ### Fix Added all missing patterns to DEFAULT_INFRASTRUCTURE. Added regression test that verifies all common project files are covered. ### Impact With this fix, the GRM-64 migration (28 changed scripts/** files + REVIEW_CHECKLIST.md) would be correctly classified as workflow-only — no unnecessary release triggered. All 822 tests pass, 100% coverage. Closes DEVX-8
emil reviewed 2026-06-22 21:06:03 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil reviewed 2026-06-22 21:11:26 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit dbe5e5e8d0 into master 2026-06-22 21:12:17 +00:00
emil deleted branch DEVX-8-fix-classification-gaps 2026-06-22 21:12:17 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#18