@platforma-sdk/block-tools 2.10.17 → 2.10.19

package/src/structure/rules/model-package-json.ts

@@ -15,6 +15,9 @@ import {
   ensureDep,
   ensureDevDep,
   ensureDevDeps,
+  removeDep,
+  removeScript,
+  removeField,
   ensurePeerDeps,
   enforceAlphabeticalOrder,
   enforceFieldOrder,
@@ -24,6 +27,7 @@ import {
 } from "../engine/api";
 import { canonicalPackageJsonOrder } from "./shared/key-order";
 import { COLOCATED_TEST_GLOB } from "./shared/colocated-tests";
+import { removeRetiredToolchainDeps } from "./shared/retired-deps";
 
 export function modelPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
   const v = ctx.blockVars;
@@ -49,8 +53,6 @@ export function modelPackageJsonInitial(ctx: RunContext): Record<string, unknown
       watch: "ts-builder build --target block-model --watch",
       build: "ts-builder build --target block-model && block-tools build-model",
       check: "ts-builder check --target block-model",
-      // No `test`: the vitest `test` script is wired by the body rule ONLY
-      // when co-located test files exist (a freshly-init'd model has none).
     },
     dependencies: {
       "@platforma-sdk/model": "sdk:",
@@ -59,7 +61,6 @@ export function modelPackageJsonInitial(ctx: RunContext): Record<string, unknown
       "@milaboratories/ts-builder": "sdk:",
       "@milaboratories/ts-configs": "sdk:",
       "@platforma-sdk/block-tools": "sdk:",
-      vitest: "catalog:",
     },
     peerDependencies: {
       "@types/node": "*",
@@ -94,7 +95,6 @@ export function modelPackageJsonRules(): void {
     "@milaboratories/ts-builder": "sdk:",
     "@milaboratories/ts-configs": "sdk:",
     "@platforma-sdk/block-tools": "sdk:",
-    vitest: "catalog:",
   });
 
   ensurePeerDeps({
@@ -102,18 +102,36 @@ export function modelPackageJsonRules(): void {
     typescript: "*",
   });
 
-  // The vitest `test` script + node ambient types are wired ONLY when the
-  // model carries co-located unit tests (`src/**/*.test.ts`). A test-less
-  // model gets neither — turbo simply has no `test` task for it, and the
-  // scope stays a refresh fixpoint. When tests ARE present, `@types/node`
-  // is promoted from peer to a devDep (the single-section invariant drops
-  // the peer) so the tests' `node:*` imports resolve under the
-  // `types: ["node"]` that model/tsconfig adds under the same predicate.
-  // Runs AFTER ensurePeerDeps so the promotion is not reverted.
-  when(whenFilesExist(COLOCATED_TEST_GLOB), () => {
-    ensureScript("test", "vitest run --passWithNoTests");
-    ensureDevDep("@types/node", "*");
-  });
+  // The vitest `test` script, the `vitest` devDep, and node ambient types are
+  // wired ONLY when the model carries co-located unit tests (`src/**/*.test.ts`).
+  // A test-less model gets none — turbo simply has no `test` task for it, and
+  // the scope stays a refresh fixpoint. When tests ARE present, `@types/node`
+  // is promoted from peer to a devDep (the single-section invariant drops the
+  // peer) so the tests' `node:*` imports resolve under the `types: ["node"]`
+  // that model/tsconfig adds under the same predicate. Runs AFTER ensurePeerDeps
+  // so the promotion is not reverted.
+  //
+  // Skipped entirely for sdk-internal (in-monorepo) blocks: those live inside
+  // the platforma monorepo's own test infrastructure (build-configs, shared
+  // catalog, coverage providers) and own their test wiring — the structurer
+  // must not impose or strip standalone-block test conventions on them.
+  when(
+    ({ ctx }) => !ctx.isSdkInternal,
+    () =>
+      when(
+        whenFilesExist(COLOCATED_TEST_GLOB),
+        () => {
+          ensureScript("test", "vitest run --passWithNoTests");
+          ensureDevDep("vitest", "catalog:");
+          ensureDevDep("@types/node", "*");
+        },
+        () => removeDep("vitest"),
+      ),
+  );
+
+  removeRetiredToolchainDeps();
+  removeField("tsup");
+  removeScript("lint");
 
   enforceAlphabeticalOrder("dependencies");
   enforceAlphabeticalOrder("devDependencies");

package/src/structure/rules/root-catalog-bump.ts

@@ -26,12 +26,14 @@ import {
   generate,
   ensureCatalogLatest,
   ensureCatalogVersion,
+  pinCatalogToDependencyOf,
 } from "../engine/api";
 import { getActiveRunContext } from "../engine/builders";
 import {
   rootPnpmWorkspaceInitial,
   SDK_CATALOG_PACKAGES,
   INFRA_CATALOG_FLOOR,
+  DERIVED_CATALOG_PINS,
   RUNENV_PYTHON,
   RUNENV_PYTHON_VERSION,
 } from "./root-pnpm-workspace";
@@ -57,8 +59,17 @@ export function rootCatalogBumpRules(): void {
               // block's missing standard keys + overwrites the init
               // placeholder).
               for (const name of SDK_CATALOG_PACKAGES) ensureCatalogLatest(name);
+              // Derived catalog pins → the EXACT version their source package
+              // declares (e.g. `vue` from `@platforma-sdk/ui-vue`). OVERWRITES
+              // (seeds when absent, tightens a loose on-disk entry). Resolved
+              // by the prefetched `derivedPinLookup`; a no-op when absent
+              // (default refresh passes none). Must run AFTER ensureCatalogLatest
+              // — the source package's own version is bumped there first.
+              for (const pin of DERIVED_CATALOG_PINS) {
+                pinCatalogToDependencyOf(pin.entry, { of: pin.of, ofVersion: pin.ofVersion });
+              }
               // Curated infra floor → fixed version, ADD-IF-ABSENT (seeds
-              // shx/turbo/vitest/vue/changesets when missing; never touches a
+              // shx/turbo/vitest/changesets when missing; never touches a
               // version the block already pins).
               for (const [name, version] of Object.entries(INFRA_CATALOG_FLOOR)) {
                 ensureCatalogVersion(name, version);

package/src/structure/rules/root-package-json.ts

@@ -8,6 +8,7 @@ import {
   ensureScript,
   ensureDevDeps,
   ensurePeerDeps,
+  removeScript,
   removeField,
   pruneKeysMatching,
   enforceAlphabeticalOrder,
@@ -34,11 +35,10 @@ export function rootPackageJsonInitial(_ctx: RunContext): Record<string, unknown
       watch: "turbo watch build",
       changeset: "changeset",
       "version-packages": "changeset version",
-      // Deprecated in favour of `update` (the full refresh → install →
-      // refresh → format flow); kept for compatibility.
+      // Deprecated alias of `upgrade-sdk`; kept for compatibility.
       "update-sdk": "block-tools structure refresh --update-deps-only",
-      update:
-        "block-tools structure refresh --update-deps-only && pnpm i && block-tools structure refresh && pnpm fmt",
+      "upgrade-sdk":
+        "block-tools structure refresh --update-deps-only && pnpm i && block-tools structure refresh && pnpm i && pnpm fmt",
     },
     peerDependencies: {
       oxlint: "*",
@@ -76,15 +76,19 @@ export function rootPackageJsonRules(): void {
   ensureScript("watch", "turbo watch build");
   ensureScript("changeset", "changeset");
   ensureScript("version-packages", "changeset version");
-  // Deprecated in favour of `update`; kept for compatibility.
+  // `update-sdk` is a deprecated alias of `upgrade-sdk`, kept for compatibility;
+  // `update` is the pre-rename name — drop it so refreshed blocks converge.
   ensureScript("update-sdk", "block-tools structure refresh --update-deps-only");
-  // Full SDK-update flow: bump catalog (deps-only) → install → re-apply
-  // structure against the freshly-pulled deps → format the result, wrapped
-  // as one script. `pnpm fmt` (turbo run fmt) leaves the tree oxfmt-clean
-  // after the structural rewrite so a follow-up `pnpm check` passes.
+  removeScript("update");
+  // Full SDK-upgrade flow: bump catalog (deps-only) → install → re-apply
+  // structure against the freshly-pulled deps → install AGAIN (the structural
+  // pass can add new devDeps — e.g. ts-builder/oxlint/oxfmt on a first
+  // migration — that `pnpm fmt` then needs on PATH) → format. Wrapped as one
+  // script. `pnpm fmt` (turbo run fmt) leaves the tree oxfmt-clean after the
+  // structural rewrite so a follow-up `pnpm check` passes.
   ensureScript(
-    "update",
-    "block-tools structure refresh --update-deps-only && pnpm i && block-tools structure refresh && pnpm fmt",
+    "upgrade-sdk",
+    "block-tools structure refresh --update-deps-only && pnpm i && block-tools structure refresh && pnpm i && pnpm fmt",
   );
 
   ensureDevDeps({

package/src/structure/rules/root-pnpm-workspace.ts

@@ -18,8 +18,9 @@
 // npm-latest for the SDK families) lives in the separate `onInitOrUpdate`
 // frame (`rootCatalogBumpRules`), which fires on init + update-deps only.
 
-import { ensureWorkspaceModulePaths, type RunContext } from "../engine/api";
-import { matchesBumpPattern } from "../engine/registry-client";
+import { ensureWorkspaceModulePaths, ensureCatalogAbsent, type RunContext } from "../engine/api";
+import { matchesBumpPattern, type DerivedCatalogPin } from "../engine/registry-client";
+import { RETIRED_TOOLCHAIN_DEPS } from "./shared/retired-deps";
 
 /** SDK packages that live in the catalog (membership only — no versions).
  *  Init fetches npm latest for each; refresh leaves them as the lockfile has
@@ -62,14 +63,23 @@ export const INFRA_CATALOG_FLOOR: Record<string, string> = {
   shx: "~0.4.0",
   "@changesets/cli": "~2.29.8",
   vitest: "~4.0.18",
-  // The seeded ui depends on `vue` (catalog:). `@platforma-sdk/ui-vue` pins
-  // vue as an EXACT regular dependency (not a peer), so the block must pin the
-  // SAME exact version — a floated `~` lets the ui resolve a newer 3.5.x than
-  // ui-vue's, yielding two vue instances and a SdkPluginV3-vs-Plugin type
-  // clash in `main.ts`. Keep this in lockstep with ui-vue's vue pin.
-  vue: "3.5.24",
 };
 
+/** Catalog entries whose exact version is DERIVED from the version another
+ *  package declares for them (see `pinCatalogToDependencyOf`). Resolved on
+ *  init + update-deps and OVERWRITES the on-disk entry.
+ *
+ *  `vue`: the seeded ui depends on `vue` (catalog:). `@platforma-sdk/ui-vue`
+ *  pins vue as an EXACT regular dependency (not a peer), so the block must
+ *  pin the SAME exact version — a floated `~` lets the ui resolve a newer
+ *  3.5.x than ui-vue's, yielding two vue instances and a SdkPluginV3-vs-Plugin
+ *  type clash in `main.ts`. Deriving the pin from ui-vue's declared dep keeps
+ *  the block in lockstep with whatever vue ui-vue's npm-latest pins, with no
+ *  hand-maintained version here. */
+export const DERIVED_CATALOG_PINS: readonly DerivedCatalogPin[] = [
+  { entry: "vue", of: "@platforma-sdk/ui-vue" },
+];
+
 /** Catalog name of the python runenv. Added to a block's catalog only when
  *  the block carries a software module. Pinned to an EXACT version — updating
  *  the python runenv is an explicit author decision, never automatic. */
@@ -100,4 +110,7 @@ export function rootPnpmWorkspaceInitial(ctx: RunContext): Record<string, unknow
 
 export function rootPnpmWorkspaceRules(): void {
   ensureWorkspaceModulePaths();
+  // Catalog side of the retired-dep cleanup — the per-package side sheds the
+  // matching `catalog:` references.
+  for (const name of RETIRED_TOOLCHAIN_DEPS) ensureCatalogAbsent(name);
 }

package/src/structure/rules/root.ts

@@ -14,7 +14,6 @@ export function rootRules(): void {
     when(
       ({ ctx }) => !ctx.isSdkInternal,
       () => {
-        // see templates/static/root/turbo.json — verbatim canonical content.
         fixed("turbo.json", file("root/turbo.json"));
         fixed(".vscode/settings.json", file("root/.vscode/settings.json"));
 

package/src/structure/rules/shared/retired-deps.ts

@@ -0,0 +1,42 @@
+// Retired toolchain dependencies — a SINGLE source of truth.
+//
+// These packages are superseded by the canonical toolchain (ts-builder owns
+// the build, vue-tsc, and the vite plugins; oxlint/oxfmt replace eslint) and
+// must be shed from a migrated block in BOTH places at once:
+//   - the root catalog          → `ensureCatalogAbsent` (root-pnpm-workspace)
+//   - every package.json scope  → `removeDep` (via removeRetiredToolchainDeps)
+//
+// Removing from only one side is a latent bug: a catalog entry dropped while a
+// package still references `<dep>: "catalog:"` leaves a dangling reference that
+// breaks `pnpm install`. Driving both from this one list makes that impossible
+// — add (or un-retire) a dependency HERE, once.
+//
+// Per-entry rationale:
+//   vite / tsup          — bundling now goes through ts-builder + rolldown.
+//   vue-tsc              — ts-builder runs vue-tsc internally for `block-ui`; a
+//                          direct dep resolves a different version than the
+//                          toolchain pins, causing type clashes in main.ts.
+//   @vitejs/plugin-vue   — bundled by ts-builder's internal vite config.
+//   vite-plugin-dts      — .d.ts emit is handled by ts-builder.
+//   @platforma-sdk/eslint-config — eslint is retired; linting runs via oxlint
+//                          inside `ts-builder check`.
+
+import { removeDep } from "../../engine/api";
+
+export const RETIRED_TOOLCHAIN_DEPS: readonly string[] = [
+  "vite",
+  "tsup",
+  "vue-tsc",
+  "@vitejs/plugin-vue",
+  "vite-plugin-dts",
+  "@platforma-sdk/eslint-config",
+];
+
+/** Drop every retired toolchain dep from the current package.json scope.
+ *  Idempotent (a `removeDep` on an absent key is a no-op), so it is safe to
+ *  call uniformly from every package.json rules body regardless of which deps
+ *  a given scope ever carried. Pairs with the catalog-side removal in
+ *  `rootPnpmWorkspaceRules`. */
+export function removeRetiredToolchainDeps(): void {
+  for (const name of RETIRED_TOOLCHAIN_DEPS) removeDep(name);
+}

package/src/structure/rules/test-package-json.ts

@@ -7,11 +7,13 @@ import {
   ensureDevDeps,
   ensurePeerDeps,
   removeDep,
+  removeScript,
   enforceAlphabeticalOrder,
   enforceFieldOrder,
   type RunContext,
 } from "../engine/api";
 import { canonicalPackageJsonOrder } from "./shared/key-order";
+import { removeRetiredToolchainDeps } from "./shared/retired-deps";
 
 export function testPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
   const v = ctx.blockVars;
@@ -21,9 +23,9 @@ export function testPackageJsonInitial(ctx: RunContext): Record<string, unknown>
     private: true,
     type: "module",
     scripts: {
-      // Type-only check (no oxlint/oxfmt — the test scope ships no lint/fmt
-      // config). Slots into the turbo `check` task.
-      check: "ts-builder type-check --target block-test",
+      fmt: "ts-builder format",
+      // `ts-builder check` runs type-check + lint (oxlint) + fmt-check (oxfmt).
+      check: "ts-builder check --target block-test",
       test: "vitest run --passWithNoTests",
     },
     peerDependencies: {
@@ -41,12 +43,13 @@ export function testPackageJsonInitial(ctx: RunContext): Record<string, unknown>
 export function testPackageJsonRules(): void {
   ensureField("type", "module");
 
-  // Type-only check (no oxlint/oxfmt — the test scope ships no lint/fmt
-  // config); slots into the turbo `check` task.
-  ensureScript("check", "ts-builder type-check --target block-test");
+  ensureScript("fmt", "ts-builder format");
+  // `ts-builder check` runs type-check + lint (oxlint) + fmt-check (oxfmt).
+  ensureScript("check", "ts-builder check --target block-test");
   ensureScript("test", "vitest run --passWithNoTests");
+  // Linting runs inside `ts-builder check` — no separate `lint` script.
+  removeScript("lint");
 
-  // @platforma-sdk/test is a test-time dep → devDependencies (matches prod).
   ensureDevDeps({
     "@platforma-sdk/test": "sdk:",
     "@milaboratories/ts-builder": "sdk:",
@@ -55,12 +58,12 @@ export function testPackageJsonRules(): void {
   });
 
   // The test tsconfig (@milaboratories/ts-configs/block/test) supplies the
-  // ambient types; only the `typescript` peer is needed. Drop any stray
-  // `@types/node` peer a legacy block declares.
+  // ambient types, so only the `typescript` peer is needed — no `@types/node`.
   ensurePeerDeps({
     typescript: "*",
   });
   removeDep("@types/node");
+  removeRetiredToolchainDeps();
 
   enforceAlphabeticalOrder("dependencies");
   enforceAlphabeticalOrder("devDependencies");

package/src/structure/rules/test.ts

@@ -11,6 +11,10 @@ export function testRules(): void {
     // lets the check pass on a test-less block while `include` still
     // type-checks real tests once present.
     fixed("tsconfig.json", file("test/tsconfig.json"));
+    // Block-local oxlint/oxfmt config — test sources are lint + fmt-checked
+    // (the `check` script runs `ts-builder check --target block-test`).
+    fixed(".oxlintrc.json", file("test/.oxlintrc.json"));
+    fixed(".oxfmtrc.json", file("test/.oxfmtrc.json"));
     // scaffold (not fixed): integration tests legitimately tune timeout /
     // retry per block. Write the canonical default if absent; never
     // overwrite an author's config.

package/src/structure/rules/ui-package-json.ts

@@ -16,6 +16,7 @@ import {
   ensurePeerDeps,
   ensureWorkspaceScopeDeps,
   removeDep,
+  removeScript,
   enforceAlphabeticalOrder,
   enforceFieldOrder,
   when,
@@ -25,6 +26,7 @@ import {
 import { scopeDepMap } from "../engine/ctx";
 import { canonicalPackageJsonOrder } from "./shared/key-order";
 import { COLOCATED_TEST_GLOB } from "./shared/colocated-tests";
+import { removeRetiredToolchainDeps } from "./shared/retired-deps";
 
 export function uiPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
   const v = ctx.blockVars;
@@ -38,8 +40,6 @@ export function uiPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
       watch: "ts-builder build --target block-ui --watch",
       build: "ts-builder build --target block-ui",
       check: "ts-builder check --target block-ui",
-      // No `test`: the vitest `test` script is wired by the body rule ONLY
-      // when co-located test files exist (a freshly-init'd ui has none).
     },
     dependencies: {
       "@platforma-sdk/ui-vue": "sdk:",
@@ -51,9 +51,9 @@ export function uiPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
     devDependencies: {
       "@milaboratories/ts-builder": "sdk:",
       "@milaboratories/ts-configs": "sdk:",
-      vitest: "catalog:",
     },
     peerDependencies: {
+      "@types/node": "*",
       typescript: "*",
     },
   };
@@ -77,35 +77,43 @@ export function uiPackageJsonRules(): void {
   ensureDevDeps({
     "@milaboratories/ts-builder": "sdk:",
     "@milaboratories/ts-configs": "sdk:",
-    vitest: "catalog:",
   });
 
-  // `vue-tsc` is owned by `ts-builder` (which runs it internally for
-  // `--target block-ui`), so the ui must NOT declare its own. A direct
-  // `vue-tsc` dep is the vestigial vite/vue-tsc-era artefact: it resolves
-  // independently of the version ts-builder pins, which is how a block ended
-  // up type-checking under a different vue-tsc than the toolchain intends.
-  removeDep("vue-tsc");
+  removeRetiredToolchainDeps();
+  removeScript("preview");
+  removeScript("lint");
 
-  // The ui builds with `--target block-ui` (types: []), so by default it
-  // carries no browser/vitest/node ambient types — only the `typescript`
-  // peer for IDE type resolution. Drop any stray `@types/node` a legacy
-  // block declares...
+  // `@types/node` is declared as a peer by default — mirrors the model scope
+  // (the block author resolves it from the workspace root). The ui builds with
+  // `--target block-ui` (types: []), so it pulls no node ambient types into the
+  // build; the peer only satisfies IDE / type resolution, alongside `typescript`.
   ensurePeerDeps({
+    "@types/node": "*",
     typescript: "*",
   });
-  removeDep("@types/node");
 
-  // ...but a ui WITH co-located unit tests (`src/**/*.test.ts`) needs the
-  // vitest `test` script AND node ambient types: re-add `@types/node` as a
-  // devDep so `types: ["node"]` in ui/tsconfig (wired under the same
-  // predicate) resolves the tests' `node:*` imports. A test-less ui keeps
-  // the lean default (no test script, no @types/node) and stays a fixpoint.
-  // Runs AFTER removeDep so the dev dep is not stripped.
-  when(whenFilesExist(COLOCATED_TEST_GLOB), () => {
-    ensureScript("test", "vitest run --passWithNoTests");
-    ensureDevDep("@types/node", "*");
-  });
+  // A ui WITH co-located unit tests (`src/**/*.test.ts`) needs the vitest
+  // `test` script, the `vitest` devDep, AND node ambient types promoted from
+  // peer to a devDep (the single-section invariant drops the peer) so that
+  // `types: ["node"]` in ui/tsconfig — wired under the same predicate —
+  // resolves the tests' `node:*` imports. A test-less ui keeps `@types/node`
+  // as a peer and carries no vitest, staying a refresh fixpoint.
+  //
+  // Skipped entirely for sdk-internal (in-monorepo) blocks: they own their test
+  // wiring under the monorepo's shared infrastructure — see model-package-json.
+  when(
+    ({ ctx }) => !ctx.isSdkInternal,
+    () =>
+      when(
+        whenFilesExist(COLOCATED_TEST_GLOB),
+        () => {
+          ensureScript("test", "vitest run --passWithNoTests");
+          ensureDevDep("vitest", "catalog:");
+          ensureDevDep("@types/node", "*");
+        },
+        () => removeDep("vitest"),
+      ),
+  );
 
   enforceAlphabeticalOrder("dependencies");
   enforceAlphabeticalOrder("devDependencies");

package/src/structure/rules/workflow-package-json.ts

@@ -8,6 +8,7 @@ import {
   ensureScript,
   ensureDep,
   ensureDevDeps,
+  removeDep,
   ensureWorkspaceScopeDeps,
   enforceAlphabeticalOrder,
   enforceFieldOrder,
@@ -18,6 +19,7 @@ import {
 import { scopeDepMaps } from "../engine/ctx";
 import { canonicalPackageJsonOrder } from "./shared/key-order";
 import { COLOCATED_TEST_GLOB } from "./shared/colocated-tests";
+import { removeRetiredToolchainDeps } from "./shared/retired-deps";
 
 export function workflowPackageJsonInitial(ctx: RunContext): Record<string, unknown> {
   const v = ctx.blockVars;
@@ -44,7 +46,8 @@ export function workflowPackageJsonInitial(ctx: RunContext): Record<string, unkn
     devDependencies: {
       "@platforma-sdk/tengo-builder": "sdk:",
       "@platforma-sdk/test": "sdk:",
-      vitest: "catalog:",
+      // `vitest` is NOT seeded here — it's wired by the body rule only when the
+      // workflow carries co-located tests (a freshly-init'd workflow has none).
       shx: "catalog:",
     },
   };
@@ -53,21 +56,33 @@ export function workflowPackageJsonInitial(ctx: RunContext): Record<string, unkn
 export function workflowPackageJsonRules(): void {
   ensureField("type", "module");
 
-  // No `fmt`: the workflow is Tengo, not TS. Build + check go through
-  // pl-tengo. shx powers the cross-platform build clean-up.
+  // No `fmt` script: the workflow is Tengo, not TS — build + check run via
+  // pl-tengo; `format` runs the emacs-batch Tengo formatter (no-op when absent).
   ensureScript("build", "shx rm -rf dist && pl-tengo build");
   ensureScript("check", "pl-tengo check");
-  // Tengo source formatter (emacs batch); no-op notice when emacs is absent.
   ensureScript("format", "/usr/bin/env emacs --script ./format.el || echo 'No emacs.'");
-  // The vitest `test` script is wired ONLY when the workflow carries
-  // co-located integration tests (`src/**/*.test.ts`, incl. `src/test/`).
-  // A test-less workflow gets no `test` task and stays a refresh fixpoint.
-  // (Workflow tests type-check via the standalone workflow/tsconfig with
-  // `types: []` — they pull their types from `@platforma-sdk/test` — so no
-  // node-types wiring is needed here, unlike model/ui.)
-  when(whenFilesExist(COLOCATED_TEST_GLOB), () => {
-    ensureScript("test", "vitest run --passWithNoTests");
-  });
+  // The vitest `test` script AND the `vitest` devDep are wired ONLY when the
+  // workflow carries co-located integration tests (`src/**/*.test.ts`, incl.
+  // `src/test/`). A test-less workflow gets neither — no `test` task, no
+  // vitest dep — and stays a refresh fixpoint. (Workflow tests type-check via
+  // the standalone workflow/tsconfig with `types: []` — they pull their types
+  // from `@platforma-sdk/test` — so no node-types wiring is needed here,
+  // unlike model/ui.)
+  //
+  // Skipped entirely for sdk-internal (in-monorepo) blocks: they own their test
+  // wiring under the monorepo's shared infrastructure — see model-package-json.
+  when(
+    ({ ctx }) => !ctx.isSdkInternal,
+    () =>
+      when(
+        whenFilesExist(COLOCATED_TEST_GLOB),
+        () => {
+          ensureScript("test", "vitest run --passWithNoTests");
+          ensureDevDeps({ vitest: "catalog:" });
+        },
+        () => removeDep("vitest"),
+      ),
+  );
 
   ensureDep("@platforma-sdk/workflow-tengo", "sdk:");
 
@@ -80,10 +95,13 @@ export function workflowPackageJsonRules(): void {
   ensureDevDeps({
     "@platforma-sdk/tengo-builder": "sdk:",
     "@platforma-sdk/test": "sdk:",
-    vitest: "catalog:",
     shx: "catalog:",
   });
 
+  // No-op for a canonical Tengo workflow; called uniformly across every scope
+  // for safety.
+  removeRetiredToolchainDeps();
+
   enforceAlphabeticalOrder("dependencies");
   enforceAlphabeticalOrder("devDependencies");
   enforceAlphabeticalOrder("peerDependencies");

package/src/structure/rules/workflow.ts

@@ -11,25 +11,49 @@ import {
   tpl,
   generate,
   remove,
+  when,
+  whenFilesExist,
   blockVars,
 } from "../engine/api";
 import { getActiveRunContext } from "../engine/builders";
 import { workflowPackageJsonInitial, workflowPackageJsonRules } from "./workflow-package-json";
+import { COLOCATED_TEST_GLOB } from "./shared/colocated-tests";
 
 export function workflowRules(): void {
   scope("workflow", () => {
-    // tsconfig for the TypeScript vitest integration tests under src/test
-    // (e.g. mixcr's workflow/src/test/columns.test.ts). types:[] keeps the
-    // isomorphic workflow free of ambient Node/DOM types.
-    fixed("tsconfig.json", file("workflow/tsconfig.json"));
     // Tengo source formatter (emacs batch); the `format` script invokes it.
     fixed("format.el", file("workflow/format.el"));
-    // scaffold (not fixed): tengo test config is author-tunable.
-    scaffold("vitest.config.mts", file("workflow/vitest.config.mts"));
+    // The TS-test tsconfig + vitest config exist ONLY when the workflow carries
+    // co-located tests (`src/**/*.test.ts`, incl. `src/test/` — e.g. mixcr's
+    // workflow/src/test/columns.test.ts). They pair with the conditional
+    // `vitest` devDep + `test` script in workflow-package-json. tsconfig is
+    // `fixed` (structural: types:[] keeps the isomorphic workflow free of
+    // ambient Node/DOM types); vitest.config is `scaffold` (author-tunable once
+    // present). A test-less workflow gets neither and stays a refresh fixpoint.
+    //
+    // Skipped entirely for sdk-internal (in-monorepo) blocks: they own their
+    // test config under the monorepo's shared infrastructure (e.g. a bespoke
+    // `vitest run --coverage` + `createVitestConfig`), which the structurer
+    // must neither scaffold nor remove. See workflow-package-json.
+    when(
+      ({ ctx }) => !ctx.isSdkInternal,
+      () =>
+        when(
+          whenFilesExist(COLOCATED_TEST_GLOB),
+          () => {
+            fixed("tsconfig.json", file("workflow/tsconfig.json"));
+            scaffold("vitest.config.mts", file("workflow/vitest.config.mts"));
+          },
+          () => {
+            remove("tsconfig.json");
+            remove("vitest.config.mts");
+          },
+        ),
+    );
     // No workflow facade (index.js / index.d.ts): the block loader resolves the
     // workflow via the `block:` components dist path (workflow/dist/.../main.plj.gz),
-    // never the JS facade. It was orphaned in production — dropped. Actively
-    // removed so existing blocks that carry it get cleaned (idempotent on absence).
+    // never the JS facade. Removed so blocks that carry one get cleaned
+    // (idempotent on absence).
     remove("index.js");
     remove("index.d.ts");
 

package/src/structure/templates/static/test/.oxfmtrc.json

@@ -0,0 +1,4 @@
+{
+  "extends": ["node_modules/@milaboratories/ts-builder/configs/oxfmt.json"],
+  "ignorePatterns": ["dist", "coverage", "CHANGELOG.md"]
+}

package/src/structure/templates/static/test/.oxlintrc.json

@@ -0,0 +1,3 @@
+{
+  "extends": ["node_modules/@milaboratories/ts-builder/dist/configs/oxlint-test.json"]
+}