test(spec): the external-binding refusal tests assert the envelope and the named subject, not the remedy's wording

Claude-Session: https://claude.ai/code/session_013dX1BHfcVSJNSkfA6cvwJN
Co-authored-by: Claude <noreply@anthropic.com>

Author: claude

packages/spec/src/data/external-binding-datasource.test.ts

@@ -37,14 +37,11 @@ describe('ObjectSchema refuses an `external` binding on the primary datasource',
     ['`datasource` omitted', { external: { remoteName: 'orders' } }],
     ["`datasource: 'default'`", { datasource: 'default', external: { remoteName: 'orders' } }],
     ['an empty binding', { external: {} }],
-  ])('%s: one issue at `external`, naming the rule and both fixes', (_label, extra) => {
+  ])('%s: one issue at `external`, naming the object and its datasource', (_label, extra) => {
     const issues = issuesOf(object(extra));
     expect(issues).toHaveLength(1);
     expect(issues[0]).toMatchObject({ code: 'custom', path: ['external'] });
     expect(issues[0]!.message).toMatch(PRIMARY);
-    expect(issues[0]!.message).toContain("ADR-0015 §4.2 allows `external` only on a datasource whose `schemaMode` is not 'managed'.");
-    expect(issues[0]!.message).toContain("declared with `schemaMode: 'external'` (or 'validate-only')");
-    expect(issues[0]!.message).toContain('or remove `external` to keep this a managed object.');
   });
 
   it('`ObjectSchema.create` refuses it at module load, with the same issue', () => {

packages/spec/src/stack-external-binding-datasource.test.ts

@@ -48,16 +48,12 @@ describe('defineStack — an `external` binding on a managed datasource the stac
   it.each([
     ["`schemaMode: 'managed'`", { schemaMode: 'managed' }],
     ['no `schemaMode` (managed by default)', {}],
-  ])('%s: STACK_CROSS_REFERENCE_INVALID, 422, one finding naming the object, the datasource and both fixes', (_label, mode) => {
+  ])('%s: STACK_CROSS_REFERENCE_INVALID, 422, one finding naming the object and the datasource', (_label, mode) => {
     const err = refusalOf({ datasources: [warehouse(mode)], objects: [order({ datasource: 'warehouse' })] });
     expect(err?.code).toBe('STACK_CROSS_REFERENCE_INVALID');
     expect(err?.status).toBe(422);
-    expect(err?.message).toMatch(/^defineStack cross-reference validation failed \(1 issue\):/);
     expect(err?.issues).toHaveLength(1);
-    const finding = String(err!.issues![0]);
-    expect(finding).toMatch(/^Object 'wh_order' declares `external` on datasource 'warehouse', whose `schemaMode` is 'managed'/);
-    expect(finding).toContain("ADR-0015 §4.2 allows `external` only on a datasource whose `schemaMode` is not 'managed'.");
-    expect(finding).toContain("declare it with `schemaMode: 'external'` (or 'validate-only')");
+    expect(String(err!.issues![0])).toMatch(/^Object 'wh_order' declares `external` on datasource 'warehouse', whose `schemaMode` is 'managed'/);
   });
 
   it('one finding per bound object; an unbound object on the same datasource is not one', () => {
@@ -101,7 +97,6 @@ describe('defineStack — an `external` binding on the primary datasource', () =
     expect(err?.issues).toEqual([
       expect.objectContaining({ code: 'custom', path: ['objects', 0, 'external'] }),
     ]);
-    expect(err?.message).toContain("objects.0.external: Object 'wh_order' declares `external`, but its `datasource` is 'default'");
   });
 
   it('a package `defaultDatasource` naming an external datasource does not make the object federated', () => {
@@ -111,6 +106,9 @@ describe('defineStack — an `external` binding on the primary datasource', () =
       objects: [order()],
     });
     expect(err?.code).toBe('STACK_SCHEMA_INVALID');
-    expect(err?.message).toContain('A package `defaultDatasource` or a `datasourceMapping` rule does not count');
+    expect(err?.status).toBe(422);
+    expect(err?.issues).toEqual([
+      expect.objectContaining({ code: 'custom', path: ['objects', 0, 'external'] }),
+    ]);
   });
 });