Epic: minor SMRT fixup batch

[willgriffin]

This was generated by AI during triage.

Purpose

Batch a small set of verified, reviewable fixes that are still present in current main as of 2026-05-26. Each item should be independently testable and safe to land in one compact cleanup branch if the diff stays small.

Quick Fix Candidates

• WorkspaceShell inspector rail overlaps content on mobile #1291 — WorkspaceShell mobile inspector rail padding: verified in packages/smrt-svelte/src/components/workspace/WorkspaceShell.svelte; the mobile media rule still resets content/topbar right padding to --smrt-ws-page-pad even when only inspectorRail is visible.
• smrt-profiles OidcIdentityCollection rejects issuer/subject at runtime #1289 — OidcIdentityCollection runtime field validation: verified in packages/profiles/src/models/OidcIdentity.ts; only profileId is decorated while provider, issuer, subject, email, and lastUsedAt are runtime-queryable fields used by collection methods.
• fix(core): stop generating test-manifest-stub.ts into src/manifest during build #1024 — legacy src/manifest/test-manifest-stub.ts generation: verified in packages/core/src/vite-plugin/index.ts; closeBundle() still writes the stub when src/manifest/ exists.
• ObjectRegistry: add qualified-name iteration API to fix cross-package collision in getClassNames-driven loops #1276 — qualified class-name iteration API: verified in packages/core/src/registry/name-resolver.ts, packages/core/src/testing/database.ts, and CLI setup utilities; getClassNames() remains simple-name/dedup oriented while collision-sensitive schema loops still need a qualified iteration surface.
• Refactor docs and release workflows to use pnpm where possible #1085 — docs/release workflow pnpm cleanup: verified docs/package-lock.json is still tracked, docs is not in pnpm-workspace.yaml, root docs scripts still call npm run --prefix docs, and publish.yml still uses npm ci --prefix docs / npm run --prefix docs build.
• smrt-svelte: navTreeFromManifest helper — derive admin nav from @smrt() metadata #1248 — navTreeFromManifest docs/closeout: implementation, export, filtering, section hints, and tests are present in packages/smrt-svelte/src/components/workspace/; the remaining quick closeout is adding/confirming consumer-facing smrt-svelte docs examples before closing the issue.

Explicitly Not In This Batch

• WorkspaceShell right tool rail should support immediate render and active-icon toggle #1293 — still valid, but it is a broader shell/tool-dock API design item; ToolsDock already covers part of the active-toggle story.
• Purge legacy compatibility paths before public release #1244 — valid pre-release cleanup, but it is an intentional breaking compatibility purge.
• Deprecate smrt docs:claude in favor of native CLAUDE.md @-imports #1178 — partially progressed (CLAUDE.md is now shipped by the noted packages), but command removal and downstream migration are broader docs/workflow cleanup.
• feat(smrt-svelte): workspace API follow-ups from first migration (ergot.io#58) #1235 — mostly shipped, but remains an umbrella for residual workspace ergonomics and hydration-ordering discussion.
• schema generator: emit UNIQUE NULLS NOT DISTINCT on PG for indexes derived from nullable conflict columns #1260 — still valid as a PG15+ schema performance/cleanliness enhancement, but correctness is already covered by @happyvertical/sql@0.74.0; generator/migration dialect work is broader than this batch.
• P2: Expose true crop/extract API in image processor adapter #1022 — still valid in SMRT source, but @happyvertical/images@0.74.1 still exposes only resize() on the processor interface; true crop/extract needs the SDK adapter API before SMRT can consume it cleanly.
• perf: batch ensureSystemTables + reduce schema passing in initialize() #972 — partially progressed: initialize() no longer passes all schemas into getDatabase(), but the remaining system-table DDL batching needs adapter/transaction semantics and is not a quick safe change.
• feat(smrt-svelte): AdminShell opinionated wrapper (deferred) #1230 — intentionally deferred placeholder for a future AdminShell wrapper after real migrations validate the primitive APIs.
• Eliminate the .smrt/manifest.json vs committed manifest.json split #1127 — still valid manifest architecture work, but it is a larger package/discovery design change rather than a small fix.
• perf(core): eager inheritedFields invalidation to retire STI save-time rehydration loop #1139 — still valid perf debt around STI inherited-field invalidation, but it touches registry/object save behavior and needs focused tests.
• [DRAFT] epic: monorepo standards audit & per-package alignment #1191 / [DRAFT] CC-2: chore(repo): normalize package.json across all packages #1193 / [DRAFT] CC-3: chore(repo): migrate all packages to createPackageConfig and standardize build #1194 / [DRAFT] CC-4: chore(repo): roll out smrtVitestPlugin and standardize test layout #1195 / [DRAFT] CC-6: chore(ci): add lefthook and CI guards to prevent standards drift #1197 / epic: Tooling/Templates packages — Phase 2 alignment #1216 — standards/template alignment work remains broader than a quick fix.

Done During Triage

Closed issues that were already implemented or superseded after source/package verification: #1226, #1227, #1228, #1229, #1247, #1246, #1214, #1189, #1152, and #1115.