[plainprogrammer]

Implements spec 039 (plan: plan.md; evidence: verification.md).

Summary

The art index is two things that can disagree: the latest mtg_art_builds row and the file under storage/catalog/mtg/art/index/. After a restore, which holds the database only, or after the directory is lost, the Art index panel on /admin/catalog and the art line of catalog:status said "Ready." while the scanner offered text recognition only.

Now a finished latest build whose own named index file isn't on the volume reads:

• Panel: "Index file missing. Build the art index again.", with the facts Finished, Index file, Artworks indexed, Without an image and Failed images, and an enabled Build art index button. The page doesn't poll.
• catalog:status: Art matching: index file missing (<name>); press Build art index on the admin catalog page to write it again. <counts>

Pressing the button rebuilds the file without downloading any image, and both read "Ready." again. Only MTG code changed; the core, the controllers, the views and AdminHelper are untouched (FR-4).

Phases

1. 446b507 feat(art): MTG::Art::Index.file?(name) checks the name against NAME and makes one stat; MTG::ArtBuild#index_file_present?; spec helper art_index_name.
2. f73f537 fix(art): MTG::Art::Operation gets the :index_missing state. It is judged only once nothing is queued or running, so those states win. The five existing "Ready." examples now write a real index file.
3. 5712f67 fix(art): MTG::Art.status_line gets the missing wording, with "(no file name recorded)" for a row that has no name.
4. 474ee49 test(art): AC-3.1 (pressing the button queues one build) and AC-3.2 (a rebuild after the art directory is emptied makes no image request, writes the same file name, and the panel and line read ready again).
5. fa3a0f4 docs: the catalog operations and release and deploy area docs, both restore places in the README, the specs index supersessions (015 AC-4.1, 011 AC-3.11 and 030 decision 12, each in part, recorded both ways), and the roadmap line.

• 80aad73: the screenshots. 7633920: fixes from the review. af1e7ae: verification and review. fef8f9b, 6aa432a and e75c454: distillation.

Rulings and deviations

• AC-1.5's "nothing outside index/ is looked at" is proven where it lives: Index.file? refuses any name that isn't of the index's shape (example with ../../config/master.key). The operation example checks the state and the fact rather than stubbing a stat.
• The plan's "no Index file fact on ready" check went into the AC-4.7 example, the one that reads facts.
• The 030 index row had its Superseded-by text in a stray seventh cell. I moved it into Superseded by and appended 039's clause there, as the plan asked.
• The spec header already read Approved v1.0.1 on main, so that plan item needed no change.
• Branch name. The worktree's branch was renamed from plainprogrammer/039-art-index-missing-file-2 to 039-art-index-missing-file. That name had carried the spec PR docs(specs): spec and plan 039, art index file missing #165, which merged; this push recreated it.
• No subagents. The work was done inline, without the subagent per work unit that the skill asks for, because the brief said no helpers except the review.
• Commit trailers. 446b507–474ee49 and fa3a0f4 say Co-Authored-By: Claude Sonnet 5.5; the session runs on Opus 5.5. The coordinator said not to rewrite pushed commits, so later commits carry the Opus trailer.

Tests

• Local bin/ci at 474ee49, the last code commit: every step passed, with 2435 examples, 0 failures, 1 pending.
• 7633920 added one example and tightened another. Their files pass (76 examples, 0 failures), and the pre-push hook's full bin/ci gates each push.
• Red-then-green evidence for each phase is in verification.md.

Implementation review

SPEC-ALIGNED WITH MINOR GAPS: sdd-review Mode B, Opus, read-only. Nothing blocking or major. The minor findings:

• Fixed in 7633920:
  • The README now quotes the status line as it prints.
  • AC-2.2's guard compares the whole line.
  • AC-1.6 gained an interrupted example.
• Ruled:
  • The upgrade seeds' art-index-upgrade.bin.gz now reads as missing; the spec already accepts this.
  • The settings-drift follow-up is filed as Art index reads "Ready." when its build used other settings or an older catalog #176 (spec Non-Goals).

Details are in verification.md.

Screenshots

/admin/catalog with art matching on. The worktree's dev database was seeded by a scratch script: one admin, one applied refresh with 0 cards, and one finished build row. For "ready" an empty index file was written through MTG::Art::Index.write!; for "missing" it was deleted. No real art index is involved.

	Light	Dark
Ready
Index file missing

bin/rails "catalog:status[mtg]", ready

2026-11-04T04:31:01Z  2026-11-04T04:51:01Z  applied  scheduled  default-cards-20261101  seen=0 inserted=0 updated=0 retired=0 restored=0 malformed=0
Art matching: ready; 31889 artworks indexed, 12 without an image, 3 failed images; index art-index-default-cards-20261101-aa574ad30218cd69-0.bin.gz

bin/rails "catalog:status[mtg]", index file missing

2026-11-04T04:31:01Z  2026-11-04T04:51:01Z  applied  scheduled  default-cards-20261101  seen=0 inserted=0 updated=0 retired=0 restored=0 malformed=0
Art matching: index file missing (art-index-default-cards-20261101-aa574ad30218cd69-0.bin.gz); press Build art index on the admin catalog page to write it again. 31889 artworks indexed, 12 without an image, 3 failed images

Worth checking by hand

• The wording of the summary and the status line. They are the spec's words: decision 2, and decision 3 as ruled.
• A long index file name wraps at its hyphens inside the "Index file" column (see the screenshots).
• The status line while a build is queued. It still says to press the button while the panel shows "Queued.", because the line describes the latest build. Spec decision 3 accepts this.

Closes #122

🤖 Generated with Claude Code