Skip to content

fix(meta): mvt-oq-02-oq-04 strings in meta.additionalFiles#21876

Merged
rjwalters merged 1 commit into
mainfrom
fix/mechanic-mvt-oq02oq04-additionalfiles-strings
Jun 1, 2026
Merged

fix(meta): mvt-oq-02-oq-04 strings in meta.additionalFiles#21876
rjwalters merged 1 commit into
mainfrom
fix/mechanic-mvt-oq02oq04-additionalfiles-strings

Conversation

@rjwalters
Copy link
Copy Markdown
Owner

Fix

Convert meta.additionalFiles rich {path, purpose} entry to a plain string so the auditor follows the companion file.

Evidence

Before:

"additionalFiles": [
  {
    "path": "Proofs/MeanValueTheoremOQ02.lean",
    "purpose": "Parent file providing taylorPolynomial..."
  }
]

After:

"additionalFiles": [
  "Proofs/MeanValueTheoremOQ02.lean"
]

Why

scripts/auditor/find-targets.ts:169 skips non-string entries:

for (const af of additionalFiles) {
  if (typeof af !== 'string') continue
  ...
}

Rich-object meta entries silently disable chain-aggregate axiom/sorry counting. The slug claims axiomCount: 1 (inherited from the parent's taylor_lagrange_remainder); with the swap the auditor will correctly aggregate 0 (main) + 1 (parent) = 1 matching the claim.

Same pattern as PR #21874 (child slug mean-value-theorem-oq-02-oq-04-oq-01) and PR #21863 (picks-theorem-oq-03).

The purpose narrative is already preserved in this slug's assumptions, originalContributions, and description fields.

Validation

Concurrent PRs touching this slug

This PR touches lines 43–50 only (the additionalFiles array). No line overlap.


Automated fix by lean-mechanic agent.

Convert the rich `{path, purpose}` entry to a plain string so the
auditor follows the parent companion file (MeanValueTheoremOQ02.lean,
which carries the inherited `taylor_lagrange_remainder` axiom that the
claimed `axiomCount: 1` already accounts for).

`scripts/auditor/find-targets.ts:169` skips non-string additionalFiles.
Same swap as PR #21874 (child slug mvt-oq-02-oq-04-oq-01) and PR #21863
(picks-theorem-oq-03 cross-polytope).

The `purpose` field's narrative is already covered by the slug's
`assumptions`, `originalContributions`, and `description` fields — no
information is lost.
@rjwalters rjwalters merged commit ced7394 into main Jun 1, 2026
@rjwalters rjwalters deleted the fix/mechanic-mvt-oq02oq04-additionalfiles-strings branch June 1, 2026 10:45
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.

1 participant