Skip to content

fix(rules): clarify null-field remediation - #211

Merged
pablontiv merged 1 commit into
masterfrom
fix/issue-200-null-field-guidance
Aug 27, 2026
Merged

fix(rules): clarify null-field remediation#211
pablontiv merged 1 commit into
masterfrom
fix/issue-200-null-field-guidance

Conversation

@pablontiv

Copy link
Copy Markdown
Owner

What

Clarify the error emitted when a child .stem tries to remove an inherited schema field with null.

The message now distinguishes scoped remedies—narrowing the declaring stem's scope.match or moving records outside its scope—from removing the declaration globally. The Rootline skill carries the same guidance.

Related issue

Closes #200

Why

The previous message named only global removal. For the common subtree-scoped case, that advice could remove governance from every record beneath the declaring stem instead of only the affected records.

How

Checklist

  • Tests pass (go test ./... -race via just test)
  • Code is clean (go vet ./...; just check reports 0 lint issues and builds successfully)
  • Changes are documented if user-facing
  • Linked to its issue with a closing keyword, or deliberately left unlinked (intermediate PR of a chain)

Additional verification: just coverage-check passes at 90.5% total coverage.

@pablontiv
pablontiv merged commit 17c08d6 into master Aug 27, 2026
11 checks passed
@pablontiv
pablontiv deleted the fix/issue-200-null-field-guidance branch August 27, 2026 22:29
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.

null-field refusal names only the tree-wide way out, not the subtree one

1 participant