Skip to content

Classify the structured review writer in the native writer ledger #1977

Closed

Description

@realmarcin

The full test run for adoption PR #1976 failed test_every_writer_is_classified: the new governed scripts/record_review.py is a YAML writer but was not entered in the native writer ledger. The run otherwise passed 3,785 tests, with 87 skips and eight deselections.

This helper creates immutable review bundles only under reviews/structured/. Its write operations do not edit kb/communities, so recording scientific curation events for those observations would be incorrect.

Use the existing _EXEMPT report-only classification with an explicit reason. Keep the writer scanner, mandatory classification check, existing owed corpus-writer backlog, and curation-history requirements unchanged. Verify the writer-ledger and actual shared-saver contract tests, followed by normal PR CI. Do not edit the canonical helper to hide it from the scanner.

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions