engineering-memory 1.0.0 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engineering-memory",
-  "version": "1.0.0",
+  "version": "1.2.0",
   "description": "Installs the Engineering Memory skill and its local MCP bridge. Sign in after installing; your organization and project are resolved from your account.",
   "license": "UNLICENSED",
   "type": "module",

package/runtime/dist/src/git/git-inspector.js

@@ -4,6 +4,7 @@ import { assertManagedPath, canonicalPath } from '../utilities/files.js';
 import { sha256, stableStringify } from '../utilities/hash.js';
 import { NativeCommandRunner } from '../utilities/process.js';
 export const temporaryScaffoldingMarker = 'ENGINEERING-MEMORY-TEMPORARY';
+export const temporaryScaffoldingPattern = `${temporaryScaffoldingMarker}:`;
 export class GitInspector {
     runner;
     constructor(runner = new NativeCommandRunner()) {
@@ -47,6 +48,7 @@ export class GitInspector {
             repoRoot: root,
             head,
             changedPaths,
+            worktreeHash: sha256(`${stableStringify(normalizedManifest)}\n`),
             diffHash: sha256(`${head ?? '<unborn>'}\n${stableStringify(normalizedManifest)}\n`),
         };
     }
@@ -303,7 +305,7 @@ export class GitInspector {
         return [...found.values()].sort((left, right) => left.path === right.path ? left.line - right.line : left.path.localeCompare(right.path));
     }
     async markerMatches(root, scope, origin) {
-        const result = await this.runner.run('git', ['grep', '--no-color', '-n', '-I', '-F', ...scope, '-e', temporaryScaffoldingMarker], { cwd: root });
+        const result = await this.runner.run('git', ['grep', '--no-color', '-n', '-I', '-F', ...scope, '-e', temporaryScaffoldingPattern], { cwd: root });
         if (result.exitCode !== 0)
             return [];
         return result.stdout

package/runtime/dist/src/mcp/tool-definitions.js

@@ -81,6 +81,13 @@ export const engineeringMemoryToolNames = [
     'auth.signin_browser',
     'auth.logout',
 ];
+const discoverableKindNames = [
+    'screen_logic',
+    'component_mapping',
+    'module_logic',
+    'data_model',
+    'api_endpoint',
+];
 const reconciliationEntry = z.object({
     resourceId: z.string().min(1),
     type: z.enum(['approved_revision', 'no_semantic_memory_change']),
@@ -193,6 +200,9 @@ export function registerEngineeringMemoryTools(server, service) {
                 'figma_mapping',
                 'figma_reference',
                 'flow_logic',
+                'module_logic',
+                'data_model',
+                'api_endpoint',
                 'current_deviation',
                 'quality_gate',
                 'task_history',
@@ -348,7 +358,7 @@ export function registerEngineeringMemoryTools(server, service) {
             newResources: z
                 .array(z.object({
                 path: z.string().min(1),
-                kind: z.enum(['screen_logic', 'component_mapping']),
+                kind: z.enum(discoverableKindNames),
                 resourceKey: z.string().min(2),
             }))
                 .optional(),
@@ -382,13 +392,7 @@ export function registerEngineeringMemoryTools(server, service) {
                 content: z.string().min(2),
                 discoveryUnits: z
                     .array(z.object({
-                    kind: z.enum([
-                        'screen_logic',
-                        'component_mapping',
-                        'module_logic',
-                        'data_model',
-                        'api_endpoint',
-                    ]),
+                    kind: z.enum(discoverableKindNames),
                     patterns: z.array(z.string().min(1)).min(1),
                 }))
                     .min(1)

package/runtime/dist/src/runtime/bridge-service.js

@@ -94,7 +94,9 @@ export class BridgeService {
                     taskKind: persistedBootstrap.taskKind,
                     mode: persistedBootstrap.mode ?? 'write',
                     repoFingerprint: repository.repoFingerprint,
-                    ...(input.mode === 'read_only' ? { baselineDiffHash: repository.git.diffHash } : {}),
+                    ...(input.mode === 'read_only'
+                        ? { baselineDiffHash: repository.git.worktreeHash }
+                        : {}),
                     checkpointIdempotencyKey: checkpointId,
                     knownRevisions: persistedBootstrap.knownRevisions,
                 }),
@@ -1009,6 +1011,7 @@ export class BridgeService {
                     changedPaths: taskMode === 'read_only' ? [] : taskChangedPaths,
                     pathChanges: pathChangeManifest,
                     diffHash: repository.git.diffHash,
+                    ...(taskMode === 'read_only' ? { worktreeHash: repository.git.worktreeHash } : {}),
                     validations,
                     pendingOutboxCount,
                     newResources: input.newResources,
@@ -1305,12 +1308,19 @@ export class BridgeService {
             const authenticated = authentication.authenticated === true;
             const state = decision?.state ?? (repository.projectId ? 'bound' : 'none');
             const client = await this.clientUpdate(authenticated);
+            const shipped = objectValue(objectValue(client)?.shippedKnowledge);
+            const incomplete = shipped && (shipped.failure !== null || Number(shipped.live) < Number(shipped.expected));
             return asJsonValue({
                 authenticated,
                 repository: publicRepository(repository),
                 decision: state,
                 projectId: repository.projectId ?? decision?.projectId ?? null,
                 client,
+                ...(incomplete
+                    ? {
+                        shippedKnowledgeWarning: 'The backend is not serving all of the knowledge this release ships. Tell the user before relying on the rules: some are missing from the running server, so a review against them is incomplete. The counts and the reason are in client.shippedKnowledge.',
+                    }
+                    : {}),
                 nextAction: entryNextAction(authenticated, state),
             });
         });
@@ -2393,6 +2403,16 @@ export function newMemoryResourceCandidate(entry, policy) {
         ]).has(extension)) {
         return [];
     }
+    const name = entry.path.toLowerCase().split('/').slice(-1)[0] ?? '';
+    if (name.endsWith('.entity.ts')) {
+        return [{ path: entry.path, kind: 'data_model' }];
+    }
+    if (name.endsWith('.controller.ts')) {
+        return [{ path: entry.path, kind: 'api_endpoint' }];
+    }
+    if (name.endsWith('.module.ts') || name.endsWith('.service.ts')) {
+        return [{ path: entry.path, kind: 'module_logic' }];
+    }
     const segments = entry.path.toLowerCase().split('/').slice(0, -1);
     const screenIndex = segments.findIndex((segment) => ['screen', 'screens', 'view', 'views'].includes(segment));
     const componentIndex = segments.findIndex((segment) => ['widget', 'widgets', 'component', 'components'].includes(segment));

package/skill/references/lifecycle.md

@@ -66,7 +66,7 @@ Send calls that do not feed each other in one batch rather than one at a time: r
 
 Checkpoints, recorded corrections and reconciliations return before the backend has them, reporting `deliveryStatus: 'pending'` with the task version they will occupy. That is a completed call, not a pending one: the journal is already durable, delivery is already under way, and `task.verify` refuses while anything remains undelivered. Do not wait for it, poll it, or send it again. Wait only for what the next step uses — a proposal's identifiers, an approval's revision, a prepared lease, and verification itself.
 
-Temporary code written to reach or force a path — a pinned state, a fixed service response, a jump straight to the screen — is allowed and expected, carries the marker `ENGINEERING-MEMORY-TEMPORARY` with its reason, and is removed before verification. `task.verify` refuses while any marker is in the tree and names every line, and the commit gate refuses while one is staged.
+Temporary code written to reach or force a path — a pinned state, a fixed service response, a jump straight to the screen — is allowed and expected, carries the marker `ENGINEERING-MEMORY-TEMPORARY` immediately followed by a colon and its reason, and is removed before verification. `task.verify` refuses while any marker is in the tree and names every line, and the commit gate refuses while one is staged.
 
 Do not write task Markdown files directly. The bridge owns event IDs, expected task versions, atomic projections, outbox state, and synchronization.