Skip to content

Commit d714469

Browse files
committed
test(metadata-protocol): the live-MySQL sys_setting probe expects the plain organization_id column
Since 7b-2 the sys_setting row-identity duplicate probe keys the plain organization_id (COALESCE(user_id, '') stays); the unit test already pins that statement. The live-MySQL cell, which runs only in the full battery, still expected the old organization_id_key alias. The preflight's key docblock named the same retired alias as its sentinel example. Claude-Session: https://claude.ai/code/session_018gewcgUaNUX9hMkxeP2HE2 Co-authored-by: Claude <noreply@anthropic.com>
1 parent 4c7dcb4 commit d714469

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

‎packages/metadata-protocol/src/migrations/runtime-index-preflight.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,8 @@ export interface RuntimeIndexDuplicateGroup {
148148
/**
149149
* The key, exactly as the migration's query projects it: a bare column
150150
* name, or `<column>_key` for a column whose NULLs the index folds into a
151-
* sentinel bucket. Neither sentinel can occur in real data, so
152-
* `organization_id_key = '__global__'` reads as "organization_id IS NULL".
151+
* sentinel bucket. No sentinel can occur in real data, so
152+
* `user_id_key = ''` reads as "user_id IS NULL".
153153
*/
154154
key: Record<string, string | null>;
155155
/** How many rows hold it. */

‎packages/metadata-protocol/src/migrations/sys-setting-identity-index.live-mysql.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ describe.skipIf(!MYSQL_URL)('#9434 the sys_setting duplicate probe on a LIVE MyS
175175
const [rows] = await conn.query(buildSysSettingDuplicateProbeSqlMysql());
176176
expect(rows).toEqual([
177177
{
178-
organization_id_key: 'org_jia',
178+
organization_id: 'org_jia',
179179
namespace: 'lifecycle',
180180
key: 'retention_overrides',
181181
scope: 'tenant',

0 commit comments

Comments
 (0)