fix(metadata-protocol): type the test helper's type as MetaRef['type'] (#7682)

CI's TypeScript Type Check went red on my push: the metadata-protocol DEBT
entry records 63 raw tsc errors and `tsc --noEmit` reported 64. The +1 was mine
and real:

  sys-metadata-repository.package-writability.test.ts(166,21): error TS2322:
  Type 'string' is not assignable to type '"object" | "view" | ... |
  "email_template"'.

`putWith`'s `opts.type` was `string`; `MetaRef['type']` is a literal union. The
suite ran green under vitest either way — this package has no `typecheck`
script, so its type surface is only judged by the ledger in CI, which is
exactly the gap that let a widened parameter through a local run.

Fixed at the source, not in the ledger: DEBT is a ratchet that may only shrink,
and raising an entry for a file added in the same PR would be the worst version
of that. `tsc --noEmit` now reports 63 for the package, matching the recorded
entry, with zero errors attributable to either file this PR touches.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AmES43BMDg4bPrxTdi5q7t

Author: claude

packages/metadata-protocol/src/sys-metadata-repository.package-writability.test.ts

@@ -64,6 +64,7 @@ import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 // `@objectstack/objectql`: objectql DEPENDS ON this package, so that import
 // would close a dependency cycle turbo rejects outright.
 import { assertEngineDeleteDispatch, assertEngineUpdateDispatch } from '@objectstack/metadata-core';
+import type { MetaRef } from '@objectstack/metadata-core';
 import { ObjectStackProtocolImplementation } from './protocol.js';
 import { SysMetadataRepository, resetEnvWritableMetadataTypes } from './sys-metadata-repository.js';
 
@@ -156,10 +157,22 @@ function makeFakeEngine() {
 
 const objectBody = { name: 'showcase_task', label: 'Task', fields: { name: { type: 'text', label: 'Name' } } };
 
-/** `put` with everything but the base fixed, so each case differs in ONE way. */
+/**
+ * `put` with everything but the base fixed, so each case differs in ONE way.
+ *
+ * `type` is `MetaRef['type']`, not `string`: that field is a literal union, and
+ * a widened `string` here is a real `tsc --noEmit` error even though `vitest`
+ * runs the file happily (this package's type surface is judged by the DEBT
+ * ledger in CI, never by the test run).
+ */
 async function putWith(
   repo: SysMetadataRepository,
-  opts: { type: string; name: string; intent: 'override-artifact' | 'runtime-only'; packageId?: string },
+  opts: {
+    type: MetaRef['type'];
+    name: string;
+    intent: 'override-artifact' | 'runtime-only';
+    packageId?: string;
+  },
 ): Promise<unknown> {
   return repo
     .put(