[amar-python]

What

The sample Migration Verification Report (merged in #216) listed the T&E re-upload defect (#215) as a known limitation. #220 fixed that, so the line was untrue for current main. This replaces the limitation with a check that demonstrates the fix, and regenerates the report at a clean code version.

Refs #59 (stays open: whether the sample meets "a Defence buyer could review it" is the product owner's call).

Changes

• scripts/sample_verification_run.py: new check 13. Re-upload the T&E file, where every row now collides on the unique name, and require status error, an unchanged registry record (same id, 6 loaded, 4 failed) and an unchanged table (6 rows). Check numbering 1-12 is unchanged. Check 11 (audit) now also states that fully failed uploads leave no audit entry.
• The stale limitation is replaced by the narrower, true one: only an identical-file re-upload was exercised; overwrite with changed content and concurrent uploads were not.
• docs/samples/*: regenerated from a real run on a fresh PostgreSQL 16 database at code version f25c94f (clean working tree). 13 of 13 pass.

The new check can fail (tested)

I temporarily reverse-applied #220's change to api/services/te_loader.py and re-ran on a fresh database. On the pre-fix loader:

Check	Result
13: re-upload changes nothing	FAIL: status ok, registry [3, 0, 10] vs original [1, 6, 4], table still 6 rows (the #215 symptom)
11: audit has one entry per successful load	FAIL: 3 entries for 2 loads (the old code wrote an audit entry for a fully failed upload)

The script exited 1. The loader change was reverted before committing; this PR does not touch api/.

Not run

CI (draft PR until checks report). No tests changed, so the test catalogue is unaffected.

---

Generated by Claude Code

[amar-python]

CI: accessibility (Playwright + axe-core) failed at Frontend unit tests (vitest): log-step-contract.test.ts reports the API emits version, schema_check and cross_upload_duplicates, which the frontend's processingLogSchema does not list.

Not this PR's. This PR changes only scripts/sample_verification_run.py and docs/samples/*. The same check is red on main at 91cffef; the steps were added by M5 (0ed50a1, #109). The guard test is doing its job: an unlisted step makes the frontend reject the whole upload response as 'Unexpected response shape'.

A fix already exists: #231 (+3 lines, CI green, mergeable) and #232 (+4 lines, still running) are two competing drafts of the same one-file change to csv.functions.ts. I have not ported either into this PR: that would widen a docs-only PR and add a third copy. Once one of them merges, this PR needs only a merge of main to go green. The other three checks passed.

---

Claude Code

---

Generated by Claude Code