chore(metadata-admin): stop surfacing metadata fields the spec dropped (framework#2377) (#2658)

Two dead, unenforced author-facing props that framework PR #3176 removes from
@objectstack/spec were still *displayed* (never enforced) in the Studio
metadata-admin — the same false affordance on the UI side. Both were read
defensively off raw docs, so this is display-only with no runtime impact:

- dataset measure `certified`: useDatasetCatalog populated a
  DatasetMeasureInfo.certified flag (+ a passthrough field in
  DatasetDefaultInspector's local Measure type) that nothing rendered. Dropped.
- agent planning `strategy`/`allowReplan`: AgentPreview's Planning rail listed
  both; narrowed KeyVals keys to ['maxIterations'] (the only live knob).

Test fixtures setting `certified` updated. No public component API change.
metadata-admin suite green (799 tests).


Claude-Session: https://claude.ai/code/session_011XchBKhKdjbhQZymt9Gn9B

Co-authored-by: Claude <noreply@anthropic.com>

Author: os-zhuang

.changeset/drop-orphaned-dead-metadata-display.md

@@ -0,0 +1,21 @@
+---
+"@object-ui/app-shell": patch
+---
+
+chore(metadata-admin): stop surfacing metadata fields the spec dropped (framework#2377)
+
+`@objectstack/spec` removes a batch of dead, unenforced author-facing metadata
+properties (ADR-0049 enforce-or-remove, framework PR #3176). Two of them were
+still *displayed* — never enforced, but shown — in the Studio metadata-admin,
+which is the same false affordance on the UI side. Both were read defensively
+off raw documents, so this is a display-only cleanup with no runtime impact:
+
+- **`dataset` measure `certified`** — `useDatasetCatalog` populated a
+  `DatasetMeasureInfo.certified` flag (and `DatasetDefaultInspector` carried it
+  in its local `Measure` type) that nothing ever rendered. Dropped both; the
+  measure picker/inspector is unchanged otherwise.
+- **`agent.planning.strategy` / `allowReplan`** — `AgentPreview`'s Planning rail
+  listed both alongside the one live knob. Narrowed the `KeyVals` keys to
+  `['maxIterations']` (the only planning field the runtime reads).
+
+Test fixtures that set `certified` were updated. No public component API change.

packages/app-shell/src/views/metadata-admin/inspectors/DatasetDefaultInspector.test.tsx

@@ -25,7 +25,7 @@ const draft = {
   object: 'opportunity',
   include: ['account'],
   dimensions: [{ name: 'region', field: 'account.region', type: 'string' }],
-  measures: [{ name: 'revenue', aggregate: 'sum', field: 'amount', certified: true }],
+  measures: [{ name: 'revenue', aggregate: 'sum', field: 'amount' }],
 };
 
 describe('DatasetDefaultInspector', () => {
@@ -51,7 +51,6 @@ describe('DatasetDefaultInspector', () => {
     const patch = onPatch.mock.calls[0][0];
     expect(patch.measures).toHaveLength(2);
     expect(patch.measures[1]).toMatchObject({ aggregate: 'sum' });
-    expect(patch.measures[1]).not.toHaveProperty('certified');
   });
 
   it('adds a dimension via onPatch', () => {

packages/app-shell/src/views/metadata-admin/inspectors/DatasetDefaultInspector.tsx

@@ -108,7 +108,6 @@ type Measure = {
   label?: string;
   aggregate?: string;
   field?: string;
-  certified?: boolean;
   format?: string;
   currency?: string;
   derived?: DerivedSpec;

packages/app-shell/src/views/metadata-admin/inspectors/ReportDefaultInspector.test.tsx

@@ -18,7 +18,7 @@ const catalog: DatasetCatalogEntry[] = [
       { name: 'close_quarter', type: 'date' },
     ],
     measures: [
-      { name: 'total_amount', aggregate: 'sum', certified: true },
+      { name: 'total_amount', aggregate: 'sum' },
       { name: 'deal_count', aggregate: 'count' },
     ],
   },

packages/app-shell/src/views/metadata-admin/previews/AgentPreview.tsx

@@ -173,7 +173,7 @@ export function AgentPreview({ name, draft }: MetadataPreviewProps) {
           <div className="border-l bg-muted/20 p-3 text-xs space-y-3">
             {planning && Object.keys(planning).length > 0 && (
               <RailBlock icon={BrainCircuit} title="Planning">
-                <KeyVals data={planning} keys={['strategy', 'maxIterations', 'allowReplan']} />
+                <KeyVals data={planning} keys={['maxIterations']} />
               </RailBlock>
             )}
             {memory && Object.keys(memory).length > 0 && (

packages/app-shell/src/views/metadata-admin/previews/useDatasetCatalog.ts

@@ -31,8 +31,6 @@ export interface DatasetMeasureInfo {
   name: string;
   /** Aggregate function (sum / avg / count / …) — display hint only. */
   aggregate?: string;
-  /** Whether the dataset author certified this measure. */
-  certified?: boolean;
 }
 
 export interface DatasetCatalogEntry {
@@ -77,7 +75,6 @@ export function toCatalogEntry(doc: Record<string, unknown>): DatasetCatalogEntr
         .map((m) => ({
           name: m.name as string,
           aggregate: typeof m.aggregate === 'string' ? (m.aggregate as string) : undefined,
-          certified: m.certified === true,
         }))
     : [];
   return { name, label: resolveLabel(doc.label, name), dimensions, measures };