engineering-memory 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engineering-memory",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "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/runtime/bridge-service.js

@@ -1799,8 +1799,11 @@ export class BridgeService {
         if (pointer.verificationIntent || pointer.closeIntent) {
             throw new Error('A clean active task snapshot is required for checkpointing');
         }
-        if (pointer.taskSlug !== input.taskSlug) {
-            throw new Error('Checkpoint task slug does not match the active task pointer');
+        // The slug names the task's journal directory, and a directory is the same
+        // one whatever case it is typed in. Rejecting KAN-1003 against kan-1003
+        // refuses a correct checkpoint over nothing.
+        if (pointer.taskSlug.toLowerCase() !== input.taskSlug.toLowerCase()) {
+            throw new Error(`Checkpoint task slug ${input.taskSlug} does not match the active task ${pointer.taskSlug}`);
         }
         return pointer;
     }
@@ -2296,7 +2299,19 @@ function assertNewResourceEvidence(candidates, evidence) {
         .map(({ path, kind }) => ({ path, kind }))
         .sort((left, right) => left.path.localeCompare(right.path));
     if (stableStringify(expected) !== stableStringify(actual)) {
-        throw new Error('New screen or shared component paths require exact memory reconciliation');
+        const expectedPaths = new Set(expected.map((entry) => entry.path));
+        const actualPaths = new Set(actual.map((entry) => entry.path));
+        const missing = expected
+            .filter((entry) => !actualPaths.has(entry.path))
+            .map((entry) => `${entry.path} (${entry.kind})`);
+        const extra = actual
+            .filter((entry) => !expectedPaths.has(entry.path))
+            .map((entry) => `${entry.path} (${entry.kind})`);
+        const details = [
+            missing.length ? `missing evidence for ${missing.join(', ')}` : '',
+            extra.length ? `evidence for paths the policy does not classify: ${extra.join(', ')}` : '',
+        ].filter(Boolean);
+        throw new Error(`New screen or shared component paths require exact memory reconciliation. ${details.length ? details.join('; ') : 'The kinds do not match the project discovery policy.'}`);
     }
 }
 function assertValidationEvidence(validations) {

package/skill/references/questionnaires.md

@@ -99,6 +99,8 @@ After `task.close`, and every time, ask the user what to do with the finished wo
 
 The two pull request options take the base branch the pull request targets, typed by the user; the head is the branch the task was done on. Never guess a base branch, and never open a pull request that was not asked for.
 
+Check that a pull request can actually be opened before offering one. If the machine has no `gh` and no token to open it with, say so in the option itself and offer what is really on the table: commit and push, and the ready-to-open compare link afterwards. Offering something you cannot deliver wastes the user's answer. Never read a credential out of a keychain or a helper to open one yourself.
+
 Once a pull request exists the work is not finished and the turn does not end there. Check whether it merges cleanly. If it does, say so with the link. If it does not, name the conflicting files and ask whether to resolve them — then wait. Resolving a conflict is a change to someone else's work and needs its own yes.
 
 ## Git Hook