Skip to content

ci: docs-only exemption skips the capability inventory's owning checker #944

Description

@forkwright

Finding

The docs-only exemption added for #775 skips the kernel job's source-level wiring-inventory check even when the changed document is the inventory itself. A malformed docs/capability-inventory.toml can therefore receive the required kernel context without the repository mechanism that validates its module coverage, witness claims, and compiled-only ownership reasons.

Evidence

PR #943 is an affirmative docs-only witness:

PR #943 itself is safe: an independent local scripts/check-wiring-inventory.sh --no-log run reports all 121 modules classified across 30 capabilities and all declared witness/source invariants valid. The gap is in admission for a future inventory edit.

Desired correction

Run the source-only half of the canonical checker for docs-only changes, preferably in the fast fmt job:

scripts/check-wiring-inventory.sh --no-log

Keep boot-log assertion in the full kernel path. Do not duplicate the checker or weaken the docs-only exemption.

Done when a docs-only change to docs/capability-inventory.toml still avoids kernel build/QEMU work but cannot merge with an unclassified/duplicate module, missing witness assertion, or ownerless compiled-only capability.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions