Skip to content

Commit 4424732

Browse files
committed
fix(metadata-protocol): the src-shipped live-MySQL testkit error drops its tracker number; changeset (stage 2)
The last metadata-protocol row leaves the prose-id ledger: the testkit's isolation error already says what it guards, so only the citation goes. The ledger is recomputed with --census-ledger and holds no metadata-protocol row. Changeset: @objectstack/metadata-protocol patch. Claude-Session: https://claude.ai/code/session_01DEvba2nBuD4tWzfq8r8NFY Co-authored-by: Claude <noreply@anthropic.com>
1 parent 730cbca commit 4424732

3 files changed

Lines changed: 31 additions & 4 deletions

File tree

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
---
2+
'@objectstack/metadata-protocol': patch
3+
---
4+
5+
metadata-protocol refusals, hints and log lines no longer cite tracker numbers; each states the reason in words
6+
7+
Clause-②: no
8+
9+
Many messages the metadata protocol shows to authors, administrators and operators ended with an
10+
issue-tracker number where the reason belonged. The number goes, and where the sentence did not
11+
already say what was decided, it now does:
12+
13+
- Refusals: `insertManyData` without an engine `insertMany` now names what that method is (the
14+
partial-success batch insert, so a bad row neither fails the whole batch nor runs the good rows'
15+
`beforeInsert` hooks twice); the unknown-metadata-type refusal says a plugin cannot declare a type
16+
because `additionalTypes` was retired, having never been read; the stored non-canonical type
17+
refusals on publish and revert say the `/meta` URL door now folds a type to its canonical spelling
18+
before it writes, so such a row predates that.
19+
- The schedule-flow `organization_id` hint says why the author's value is the only source: the engine
20+
fills only an organization the run resolved, and a schedule resolves none.
21+
- Log lines: the three `kernel:ready` "migration skipped" warnings now say what the migration that did
22+
not run would have ensured; the history-counter abort says the old path took a failed read for an
23+
empty table; the publish-closure degrade says the batch's own drafts are left out of the closure;
24+
the cold-boot org-scoped audit calls the write refusal it points at declared-types-only. The
25+
overlay, `sys_view_definition` and `sys_setting` index messages, the seed/API tenancy repair and its
26+
receipt, the batch-row withhold and the object-existence gate's no-registry warning lose only the
27+
citation, because their sentences already said it.
28+
- The live-MySQL testkit's isolation error loses its citation.
29+
30+
Text only: no error code, field name, status or behaviour changes.

‎packages/metadata-protocol/src/migrations/live-mysql-database.testkit.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ export function currentLiveMysqlDatabase(): string {
216216
const testPath = expect.getState().testPath;
217217
if (!testPath) {
218218
throw new Error(
219-
'live-mysql isolation (#10382): vitest reported no testPath, so this live connection ' +
219+
'live-mysql isolation: vitest reported no testPath, so this live connection ' +
220220
'cannot be given a per-file database and would fall back to sharing one with every ' +
221221
'other live file in this package — including its `drop database` in afterAll. Call ' +
222222
'currentLiveMysqlDatabase() from a test file.',

‎scripts/doc-authoring-prose-id.baseline.json‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -297,9 +297,6 @@
297297
"#6603": 1,
298298
"#7020": 1
299299
},
300-
"packages/metadata-protocol/src/migrations/live-mysql-database.testkit.ts": {
301-
"#10382": 1
302-
},
303300
"packages/metadata/src/endpoint-matcher.ts": {
304301
"#5040": 1
305302
},

0 commit comments

Comments
 (0)