Restore Session Inspector as a first-class tool without reintroducing tool communication coupling - PR_26128_002-restore-session-inspector

Author: DavidQ

docs/dev/reports/session_inspector_restore.md

@@ -6,6 +6,7 @@
 - Added first-class registry wiring in `tools/toolRegistry.js`.
 - Added tools index grouping for `session-inspector` as a Viewer in `tools/renderToolsIndex.js`.
 - Added shared navigation grouping for `session-inspector` as a Viewer in `tools/shared/platformShell.js`.
+- Rerun delta adds an explicit Session Inspector runtime contract in `tools/session-inspector/js/services/SessionInspectorRuntimeContract.js` and wires it through tool boot status.
 
 ## Boundaries
 - No cross-tool communication was added.
@@ -17,9 +18,12 @@
 ## Validation
 - `npm run test:workspace-v2`: Pass, 10 passed.
 - Targeted Session Inspector launch test: Pass.
+- Targeted Session Inspector runtime contract validation: Pass.
 - Targeted tools index Session Inspector card/link validation: Pass.
 - Targeted Workspace Manager V2 launch validation: Pass.
 - Targeted Preview Generator V2 launch validation: Pass.
+- `tools/session-inspector/**` file existence check: Pass.
+- Git status includes actual runtime/tool changes: Pass, `tools/session-inspector/js/SessionInspectorApp.js`, `tools/session-inspector/js/bootstrap.js`, and `tools/session-inspector/js/services/SessionInspectorRuntimeContract.js`.
 - Full samples smoke test: Skipped by BUILD instruction. This PR is scoped to restoring Session Inspector first-class launch and registry/nav wiring only.
 
 ## Changed Files
@@ -34,6 +38,7 @@
 - `tools/session-inspector/js/controls/EntryListControl.js`
 - `tools/session-inspector/js/controls/FilterControl.js`
 - `tools/session-inspector/js/controls/StatusLogControl.js`
+- `tools/session-inspector/js/services/SessionInspectorRuntimeContract.js`
 - `tools/session-inspector/js/services/SessionInspectorStorageService.js`
 - `tools/toolRegistry.js`
 - `tools/renderToolsIndex.js`

tools/session-inspector/js/SessionInspectorApp.js

@@ -5,6 +5,7 @@ export class SessionInspectorApp {
     entryList,
     filters,
     refreshButton,
+    runtimeContract,
     statusLog,
     storageService
   }) {
@@ -14,6 +15,7 @@ export class SessionInspectorApp {
     this.entryList = entryList;
     this.filters = filters;
     this.refreshButton = refreshButton;
+    this.runtimeContract = runtimeContract || { storageAccess: "read-only" };
     this.statusLog = statusLog;
     this.storageService = storageService;
     this.selectedId = "";
@@ -30,7 +32,7 @@ export class SessionInspectorApp {
     });
     this.refreshButton.addEventListener("click", () => this.refresh());
     this.refresh({ silent: true });
-    this.statusLog.ok("Session Inspector ready. Storage is read-only.");
+    this.statusLog.ok(`Session Inspector ready. Storage is ${this.runtimeContract.storageAccess}.`);
   }
 
   refresh({ silent = false } = {}) {

tools/session-inspector/js/bootstrap.js

@@ -4,6 +4,7 @@ import { DetailsControl } from "./controls/DetailsControl.js";
 import { EntryListControl } from "./controls/EntryListControl.js";
 import { FilterControl } from "./controls/FilterControl.js";
 import { StatusLogControl } from "./controls/StatusLogControl.js";
+import { sessionInspectorRuntimeContract } from "./services/SessionInspectorRuntimeContract.js";
 import { SessionInspectorStorageService } from "./services/SessionInspectorStorageService.js";
 
 function requireElement(selector) {
@@ -29,6 +30,7 @@ window.addEventListener("DOMContentLoaded", () => {
       summary: requireElement("#sessionSummary")
     }),
     refreshButton: requireElement("#refreshSessionButton"),
+    runtimeContract: sessionInspectorRuntimeContract(),
     statusLog: new StatusLogControl({
       clearButton: requireElement("#clearStatusButton"),
       output: requireElement("#statusLog")

tools/session-inspector/js/services/SessionInspectorRuntimeContract.js

@@ -0,0 +1,11 @@
+const SESSION_INSPECTOR_RUNTIME_CONTRACT = Object.freeze({
+  communicatesWithTools: false,
+  storageAccess: "read-only",
+  toolId: "session-inspector",
+  usesRepoSelection: false,
+  writesStorage: false
+});
+
+export function sessionInspectorRuntimeContract() {
+  return { ...SESSION_INSPECTOR_RUNTIME_CONTRACT };
+}