Skip to content

An import that reports fifteen failures and can name ten - #216

Merged
dmitrat merged 1 commit into
mainfrom
studio-phase7
Aug 19, 2026
Merged

dmitrat merged 1 commit into
mainfrom
studio-phase7

Conversation

@dmitrat

@dmitrat dmitrat commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Phase 7 of the fix plan: import and export. Studio 1000 green, 989 before.

Finding 27 was half built, which is why it was reported as absent

The CSV path has kept every rejection since WS-36, and the window has a button that writes them
beside the source with the line, the reason and the text. The JSON path filled the display list and
recorded nothing
— so a JSON import really did report 15 failed, name ten, and have nothing to
save. Both paths record now, and the rule counts recording sites against display sites rather than
trusting either.

finding fix
25 0 columns for every file, because the count asked ColumnMappings — built on step 3. It reads the parse now, and the rows themselves are drawn: a wrong delimiter is obvious in the rows and invisible in a count
26 the preview banner and the error banner were both Grid.Row="5", so the result landed across the line saying what the file held. Two borders in one cell are drawn over each other at full opacity, in every state — the same shape as the conflict message in Phase 4
28 the collision policy moved to the step it is about; Columns held two unrelated decisions while Destination held one control
32 the export's first frame said 0 / 0 rows at 0.0 %, because the panel opens before the rows are fetched. The numbers wait for the total
4 the output path can be typed as well as browsed for
5 Export table and Export query results, in the title bar and in the Tools menu — only one of them carries the three scopes with their counts

The census of hard-coded values went red again at 69 against 68 (the preview's MaxHeight); the
ledger records it with its reason — a box round a block of content, so the wizard does not grow with
the file.

🤖 Generated with Claude Code

Phase 7 of the fix plan: import and export. Studio 1000 green, 989 before.

**Finding 27 was half built, which is why it was reported as absent.** The CSV path has kept every
rejection since WS-36, and the window has a button that writes them beside the source with the line,
the reason and the text. The JSON path filled the display list and recorded nothing - so an import of
a JSON file really did report "15 failed", name ten, and have nothing to save. Both paths record now,
and the rule counts the recording sites against the display sites rather than trusting either.

**Step 1 said "0 columns" for every file** because the count asked ColumnMappings, which is built on
step 3. It reads the parse now - and the rows themselves are drawn, which is what a preview is for: a
wrong delimiter is obvious in the rows and invisible in a count.

**The result of an import was painted across the line saying what the file held**, because the preview
banner and the error banner were both Grid.Row="5". Two borders in one cell are drawn over each other
at full opacity, in every state - the same shape as the conflict message in phase 4. The cell holds a
column now.

**The collision policy moved to the step it is about.** It sat on Columns beside the mapping, so that
step held two unrelated decisions while Destination held one control.

**The export's first frame said "0 / 0 rows" at "0.0 %"**, because the panel opens when the export
starts and the rows are fetched after that. Zero out of zero is not a small number; it is no number,
and the numbers wait for the total now.

**Two dialogs called Export Data are two dialogs.** Export table and Export query results, in the
title bar and in the Tools menu - only one of them carries the three scopes with their counts, and
somebody looking in Tools for them would not have found them. And the output path can be typed as well
as browsed for.

The census of hard-coded values went red again, at 69 against 68: the import preview's MaxHeight. The
ledger records it with its reason - a box round a block of content, so the wizard does not grow with
the file.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@dmitrat
dmitrat merged commit 73e7439 into main Aug 19, 2026
1 check passed
@dmitrat
dmitrat deleted the studio-phase7 branch August 19, 2026 07:06
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