Skip to content

Record media integrity in the State table - #48

Merged
ptr727 merged 1 commit into
developfrom
record-media-integrity-state
Aug 8, 2026
Merged

Record media integrity in the State table#48
ptr727 merged 1 commit into
developfrom
record-media-integrity-state

Conversation

@ptr727

@ptr727 ptr727 commented Aug 7, 2026

Copy link
Copy Markdown
Owner

One row. The State table answers "where is this" for someone arriving cold, and it had no row for the surface that took the most work to establish. A reader would have found the media question discussed in checks/README.md, in OPERATIONS.md, and in this file's own sections, and unanswered in the one place that summarizes.

Rebased onto develop and reconciled, having sat open since 2026-08-07. The row is not the one this branch originally proposed, for two reasons.

| Media integrity | closed for the files themselves. The conversion's only loss was 19 images,
  inside five galleries, restored from the capture, and the 98 files with no page links are
  adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either
  way. How media was *presented* was a separate loss, found later and now gated |

The claim is narrowed from "closed" to "closed for the files themselves." #55 later found that the conversion had also flattened how galleries present media, in three shapes across three posts: eleven set captions written as trailing text, two bare markdown images, and eight linked ones. None of that was a file loss and no file was missing, which is exactly why nothing saw it — every media check reasons about a URL, and misplaced content resolves and is linked just as correct content does. Saying "closed" flat would have been true of the files and misleading about the surface.

The Content and media row is left alone. This branch originally rewrote it to carry "514 pages, 778 media files". develop has since deliberately replaced counts that content additions move with what they represent, keeping only the counts that describe the legacy contract. Reintroducing a page count would have reverted that decision, so the conflict is resolved in develop's favour and only the new row is added.

The row states the conclusion rather than the activity, and names ORPHANED_MEDIA because the useful part for a later reader is that the number is defended in both directions now, not that it was once measured.

Verification

markdownlint-cli2 and editorconfig-checker clean. Documentation only — no code, config, workflow, or content change.

🤖 Generated with Claude Code

Copilot AI lite review requested due to automatic review settings August 7, 2026 03:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the project’s high-level “State” table to include a concise, durable summary of the media-integrity outcome so a new reader can find the conclusion in the central status summary.

Changes:

  • Add a new “Media integrity” row to the State table in TODO.md.
  • Document the defended ORPHANED_MEDIA invariant and the adjudicated orphaned-media count.

Comment thread TODO.md Outdated
Copilot AI review requested due to automatic review settings August 8, 2026 13:06
@ptr727
ptr727 force-pushed the record-media-integrity-state branch from 59a4cae to 071f58c Compare August 8, 2026 13:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

TODO.md:12

  • The phrase "the 98 files no page links" is grammatically incomplete and reads like a missing preposition, which makes the row harder to parse. Consider changing it to "the 98 files with no page links".
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |

@ptr727
ptr727 requested a lite review from Copilot August 8, 2026 13:10
@ptr727
ptr727 force-pushed the record-media-integrity-state branch from 071f58c to f16096e Compare August 8, 2026 13:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

TODO.md:12

  • The phrase “the 98 files no page links” is grammatically incorrect/missing a word; it reads like “with no page links”. This affects readability of the State table row.
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |

Copilot AI review requested due to automatic review settings August 8, 2026 13:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

TODO.md:12

  • This sentence uses a bare number ("loss was 19") which can be read as files/bytes/etc. Consider explicitly stating "19 images" (matching the PR description) to make the unit unambiguous in this summary table.
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |

The table said nothing about whether the conversion lost anything, which is
the one question a migration's record has to answer and the one a reader
cannot check for themselves.

Rebased onto develop and reconciled with two things that landed while this sat
open. The volatile counts it originally carried in the Content and media row
are gone, because develop deliberately replaced counts that content additions
move with what they represent, and this row would have reintroduced 514 pages
against that decision. And the claim is narrowed from "closed" to closed for
the files themselves: the image loss was 19 and is restored, but #55 later
found the conversion had also flattened how galleries present media, in three
shapes across three posts. That was not a file loss and no file was missing,
which is exactly why nothing saw it for so long, and it is now gated.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 8, 2026 13:15
@ptr727
ptr727 force-pushed the record-media-integrity-state branch from f16096e to 4cde232 Compare August 8, 2026 13:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

TODO.md:12

  • The added table row’s wording doesn’t match the PR description’s proposed "Media integrity" State-table entry (it adds extra qualifiers and an additional sentence about presentation/gating). Either update the PR description or adjust the row text so the State table remains the single-source summary as described.
| Media integrity | closed for the files themselves. The conversion's only loss was 19 images, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |

@ptr727
ptr727 requested a lite review from Copilot August 8, 2026 13:19
@ptr727

ptr727 commented Aug 8, 2026

Copy link
Copy Markdown
Owner Author

Answering the suppressed finding, which carried no thread.

The added table row's wording doesn't match the PR description's proposed "Media integrity" State-table entry […] Either update the PR description or adjust the row text

Correct, and the description was the stale half. This branch was rebased onto develop and the row was reworded in the process, while the body still quoted the original. That round ran before the body was updated.

Taking the first of your two options: the description now quotes the row verbatim and explains both changes to it.

  • "closed" narrowed to "closed for the files themselves"Restore gallery captions and items the conversion flattened #55 later found the conversion had also flattened how galleries present media, in three shapes across three posts. No file was missing, which is why nothing caught it, so a flat "closed" would have been true of the files and misleading about the surface.
  • The Content and media row is no longer touched — this branch originally rewrote it to carry "514 pages, 778 media files", and develop has since deliberately dropped counts that content additions move. Reintroducing a page count would have reverted that decision.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 466bf43 into develop Aug 8, 2026
6 checks passed
@ptr727
ptr727 deleted the record-media-integrity-state branch August 8, 2026 13:37
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.

2 participants