claude-mcp-workflow 0.1.6 → 0.1.8

package/build/types.js

@@ -98,4 +98,5 @@ export const WorkflowSessionsInputSchema = z.object({}).optional();
 export const MAX_STACK_DEPTH = 10;
 export const DEFAULT_MAX_TRANSITIONS = 50;
 export const LOCK_STALE_MS = 5000;
+export const KEEP_TERMINAL_SESSIONS = 3;
 //# sourceMappingURL=types.js.map

package/build/types.js.map

@@ -1 +1 @@
-{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,iDAAiD;AAEjD,0EAA0E;AAC1E,mFAAmF;AACnF,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC5C,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC7B,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IAC5C,QAAQ,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAChC,OAAO,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,QAAQ,EAAE;IAChE,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;IAClD,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAE3B,4DAA4D;IAC5D,MAAM,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IAEtC,mDAAmD;IACnD,iBAAiB,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAEzC,sBAAsB;IACtB,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC,QAAQ,EAAE;IAEpD,OAAO;IACP,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC1B,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;IACzC,GAAG,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACpC,UAAU,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAClC,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE,EAAE,yDAAyD;IAE7G,QAAQ;IACR,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC1B,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC,QAAQ,EAAE;IAC5E,OAAO,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACxC,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAE3B,wBAAwB;IACxB,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACjC,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACtC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,cAAc,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACrC,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,EAAE,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC,QAAQ,EAAE;IACvG,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE,EAAE,wFAAwF;CACvI,CAAC,CAAC;AAIH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IACvB,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC1B,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACnE,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,qBAAqB,CAAC;CACxC,CAAC,CAAC;AAkEH,6BAA6B;AAE7B,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,uDAAuD,CAAC;IACjG,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;IACrF,iBAAiB,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,0CAA0C,CAAC;CAC9F,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,6BAA6B,GAAG,CAAC,CAAC,MAAM,CAAC;IACpD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,gCAAgC,CAAC;IACjE,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;CACtF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;CACtD,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,6BAA6B,GAAG,CAAC,CAAC,MAAM,CAAC;IACpD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,aAAa,CAAC;IACvC,KAAK,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,CAAC,eAAe,CAAC;IAC5C,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;CACtF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,SAAS,EAAE,qBAAqB,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,yCAAyC,CAAC;IAC5H,cAAc,EAAE,CAAC,CAAC,MAAM,CAAC;QACvB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE;KACf,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACzD,iBAAiB,EAAE,CAAC,CAAC,MAAM,CAAC;QAC1B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;KACjB,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;CAC9C,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,kCAAkC,CAAC;IACpE,UAAU,EAAE,CAAC,CAAC,MAAM,CAAC;QACnB,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;QAC1B,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;QACvD,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,qBAAqB,CAAC;KACxC,CAAC,CAAC,QAAQ,CAAC,0BAA0B,CAAC;CACxC,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;CACtD,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,uBAAuB,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC;AAE/D,MAAM,CAAC,MAAM,2BAA2B,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC;AAEnE,oBAAoB;AAEpB,MAAM,CAAC,MAAM,eAAe,GAAG,EAAE,CAAC;AAClC,MAAM,CAAC,MAAM,uBAAuB,GAAG,EAAE,CAAC;AAC1C,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC"}
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,iDAAiD;AAEjD,0EAA0E;AAC1E,mFAAmF;AACnF,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC5C,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC7B,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IAC5C,QAAQ,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAChC,OAAO,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,QAAQ,EAAE;IAChE,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;IAClD,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAE3B,4DAA4D;IAC5D,MAAM,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IAEtC,mDAAmD;IACnD,iBAAiB,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAEzC,sBAAsB;IACtB,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC,QAAQ,EAAE;IAEpD,OAAO;IACP,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC1B,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;IACzC,GAAG,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACpC,UAAU,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;IAClC,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE,EAAE,yDAAyD;IAE7G,QAAQ;IACR,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC1B,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC,QAAQ,EAAE;IAC5E,OAAO,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACxC,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAE3B,wBAAwB;IACxB,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACjC,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;IACtC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC9B,cAAc,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACrC,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,EAAE,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC,QAAQ,EAAE;IACvG,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE,EAAE,wFAAwF;CACvI,CAAC,CAAC;AAIH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IACvB,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC1B,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACnE,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,qBAAqB,CAAC;CACxC,CAAC,CAAC;AAkEH,6BAA6B;AAE7B,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,uDAAuD,CAAC;IACjG,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;IACrF,iBAAiB,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,0CAA0C,CAAC;CAC9F,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,6BAA6B,GAAG,CAAC,CAAC,MAAM,CAAC;IACpD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,gCAAgC,CAAC;IACjE,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;CACtF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;CACtD,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,6BAA6B,GAAG,CAAC,CAAC,MAAM,CAAC;IACpD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,GAAG,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,aAAa,CAAC;IACvC,KAAK,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,CAAC,eAAe,CAAC;IAC5C,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,8CAA8C,CAAC;CACtF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;IACrD,SAAS,EAAE,qBAAqB,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,yCAAyC,CAAC;IAC5H,cAAc,EAAE,CAAC,CAAC,MAAM,CAAC;QACvB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE;KACf,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACzD,iBAAiB,EAAE,CAAC,CAAC,MAAM,CAAC;QAC1B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;KACjB,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;CAC9C,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,CAAC,MAAM,CAAC;IAChD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,kCAAkC,CAAC;IACpE,UAAU,EAAE,CAAC,CAAC,MAAM,CAAC;QACnB,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAClC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;QAC1B,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;QACvD,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,qBAAqB,CAAC;KACxC,CAAC,CAAC,QAAQ,CAAC,0BAA0B,CAAC;CACxC,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/C,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,oBAAoB,CAAC;CACtD,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,uBAAuB,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC;AAE/D,MAAM,CAAC,MAAM,2BAA2B,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,EAAE,CAAC;AAEnE,oBAAoB;AAEpB,MAAM,CAAC,MAAM,eAAe,GAAG,EAAE,CAAC;AAClC,MAAM,CAAC,MAAM,uBAAuB,GAAG,EAAE,CAAC;AAC1C,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC;AAClC,MAAM,CAAC,MAAM,sBAAsB,GAAG,CAAC,CAAC"}

package/hooks/workflow-cleanup.sh

@@ -1,28 +1,38 @@
 #!/bin/bash
-# SessionEnd hook: abandon active workflow sessions for this Claude Code instance.
+# SessionEnd hook: abandon this Claude Code instance's active workflow sessions
+# (and their descendants), then cap retained terminal sessions so the state
+# dir keeps only the most recent few for history — never grows unbounded.
 # Skips sessions referenced in plan files (they should survive context clears).
-# Modifies JSON files directly — does NOT depend on MCP server / dashboard being alive.
+# Modifies JSON files directly — does NOT depend on MCP server being alive.
 read -t 1 -r INPUT
 STATE_DIR="${STATE_DIR:-$HOME/.claude/workflow-state}"
 PLANS_DIR="$HOME/.claude/plans"
 NOW=$(date -u +"%Y-%m-%dT%H:%M:%S.000Z")
+# Mirror of KEEP_TERMINAL_SESSIONS in src/types.ts
+KEEP_TERMINAL=3
 ABANDONED_SIDS=()
 
-for f in "$STATE_DIR"/*.json; do
-  [ -f "$f" ] || continue
-  SID=$(jq -r "select(.stack | length > 0) | select(.context.claude_code_pid == $PPID) | .session_id" "$f" 2>/dev/null)
-  [ -n "$SID" ] || continue
-  # Skip if session referenced in plan files
-  [ -d "$PLANS_DIR" ] && grep -rql "$SID" "$PLANS_DIR"/ 2>/dev/null && continue
-  # Abandon: clear stack, set outcome, append history entry
-  TMP="${f}.tmp"
-  jq --arg now "$NOW" '
-    .history += [{ frame: .active_frame, event: "abandon", at: $now }] |
+is_planned() {
+  [ -d "$PLANS_DIR" ] && [ -n "$1" ] && grep -rqlF "$1" "$PLANS_DIR"/ 2>/dev/null
+}
+
+abandon() {
+  local f="$1" event="${2:-abandon}" TMP="${f}.tmp"
+  jq --arg now "$NOW" --arg event "$event" '
+    .history += [{ frame: .active_frame, event: $event, at: $now }] |
     .stack = [] |
     .active_frame = -1 |
     .outcome = "abandoned" |
     .updated_at = $now
   ' "$f" > "$TMP" 2>/dev/null && mv "$TMP" "$f" || rm -f "$TMP"
+}
+
+for f in "$STATE_DIR"/*.json; do
+  [ -f "$f" ] || continue
+  SID=$(jq -r "select(.stack | length > 0) | select(.context.claude_code_pid == $PPID) | .session_id" "$f" 2>/dev/null)
+  [ -n "$SID" ] || continue
+  is_planned "$SID" && continue
+  abandon "$f"
   ABANDONED_SIDS+=("$SID")
 done
 
@@ -34,14 +44,8 @@ cascade_abandon() {
     local child_sid
     child_sid=$(jq -r --arg pid "$parent_sid" 'select(.parent_session_id == $pid) | select(.stack | length > 0) | .session_id' "$f" 2>/dev/null)
     [ -n "$child_sid" ] || continue
-    TMP="${f}.tmp"
-    jq --arg now "$NOW" '
-      .history += [{ frame: .active_frame, event: "cascade_abandon", at: $now }] |
-      .stack = [] |
-      .active_frame = -1 |
-      .outcome = "abandoned" |
-      .updated_at = $now
-    ' "$f" > "$TMP" 2>/dev/null && mv "$TMP" "$f" || rm -f "$TMP"
+    is_planned "$child_sid" && continue
+    abandon "$f" cascade_abandon
     cascade_abandon "$child_sid"
   done
 }
@@ -49,3 +53,33 @@ cascade_abandon() {
 for sid in "${ABANDONED_SIDS[@]}"; do
   cascade_abandon "$sid"
 done
+
+# Cap retained terminal sessions: keep the KEEP_TERMINAL with the newest
+# .updated_at, delete older ones. Full parity with Storage.pruneTerminal in
+# src/storage.ts:
+#   - sort by the JSON .updated_at field (not file mtime) — abandon() bumps
+#     mtime via mv, so mtime order would diverge from the engine's retention;
+#   - secondary sort by .session_id — cascade abandon stamps every child with
+#     one identical .updated_at, so a stable tiebreak keeps the retained set
+#     deterministic and identical to the engine;
+#   - no is_planned guard here — terminal (empty-stack) sessions can never be
+#     plan-referenced (plan resume requires an active stack), same invariant
+#     the engine's pruneTerminal documents and relies on.
+TERM_FILES=()
+for f in "$STATE_DIR"/*.json; do
+  [ -f "$f" ] || continue
+  jq -e '.stack | length == 0' "$f" >/dev/null 2>&1 && TERM_FILES+=("$f")
+done
+if [ "${#TERM_FILES[@]}" -gt "$KEEP_TERMINAL" ]; then
+  i=0
+  while IFS=$'\t' read -r _ _ f; do
+    i=$((i + 1))
+    [ "$i" -le "$KEEP_TERMINAL" ] && continue
+    rm -f "$f"
+  done < <(
+    for f in "${TERM_FILES[@]}"; do
+      meta=$(jq -r '[.updated_at // "", .session_id // ""] | @tsv' "$f" 2>/dev/null)
+      printf '%s\t%s\n' "$meta" "$f"
+    done | sort -r -t $'\t' -k1,2
+  )
+fi

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mcp-workflow",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Structured workflow orchestration for AI agents via finite-state machines",
   "author": "AxGord <axgord@gmail.com> (https://github.com/AxGord)",
   "license": "MIT",

package/templates/bug-fix.yaml

@@ -17,9 +17,17 @@ states:
       - `visual` → UI/layout bug — requires debug bridge, screenshots, visual reproduction
       - `logic` → data/logic bug — reproducible via unit tests or logs
     transitions:
-      visual: reproduce_visual
+      visual: load_skills_visual
       logic: reproduce_logic
 
+  load_skills_visual:
+    skills:
+      - coding-skill-selector
+      - ?debug-bridge
+      - ?project-skill-selector
+    transitions:
+      continue: reproduce_visual
+
   reproduce_visual:
     task: "Reproduce the bug visually"
     prompt: |
@@ -28,17 +36,23 @@ states:
       DO NOT fix the bug here. You may add trace/log statements to help reproduce,
       but do NOT modify production logic. The fix belongs in the `fix` state.
 
-      MANDATORY FIRST ACTION — read the skill descriptions in the system-reminder skills list
-      and call `Skill(name)` for EVERY skill whose description relates to debug bridges,
-      interactive debugging, UI interaction, or the current app's domain.
-      Do NOT skip this. Working without relevant skills leads to wasted time and errors.
+      REPRODUCTION TOOLING — find the right tool:
+      1. Check available MCP tools for browser/screenshot/debug-bridge capabilities.
+      2. Pick the tool matching the platform.
+      3. Persist the choice for verify_visual:
+         `context_set({ key: "chosen_tool", value: "<short name + how to launch>" })`
+
+      "No tooling available" is almost never true for a visual app. Verify the MCP
+      tool list before choosing `cant_reproduce`. Do NOT `abort()` to escape this
+      gate — use `cant_reproduce` if genuinely blocked.
 
       MANDATORY STEPS (all required, in order):
       1. Understand the exact user path that triggers the bug (which button, which file, which action)
-      2. Build and launch the app with debug bridge
-      3. Follow the SAME path the user described to trigger the bug
-      4. Take a screenshot and VERIFY the bug is visible
-      5. Describe what you see — compare with the user's report
+      2. Build (if applicable — native targets must be built before launch)
+      3. Launch the app and connect with the tool chosen above
+      4. Follow the SAME path the user described to trigger the bug
+      5. Take a screenshot and VERIFY the bug is visible
+      6. Describe what you see — compare with the user's report
 
       STOP — self-check before transitioning:
       - Did you LAUNCH the app and take a SCREENSHOT? If not, you haven't reproduced.
@@ -50,7 +64,9 @@ states:
       `cant_reproduce` = you tried but can't see it. This leads to asking the user for help.
       NEVER use `reproduced` when you haven't seen the bug yourself.
 
-      If the bug requires file operations, use the tm-interactive-debug skill recipes.
+      If the bug requires simulated input or file operations, use the chosen tool's
+      recipes: Playwright `browser_evaluate` / `browser_press_key` for web,
+      tm-interactive-debug for native.
     transitions:
       reproduced: diagnose
       cant_reproduce: clarify
@@ -86,7 +102,7 @@ states:
       transition `try_visual` to reproduce via debug bridge instead.
     transitions:
       reproduced: diagnose
-      try_visual: reproduce_visual
+      try_visual: load_skills_visual
       cant_reproduce: clarify
 
   diagnose:
@@ -279,19 +295,33 @@ states:
       - Visual bug → `visual` (needs screenshot verification)
       - Logic bug → `logic` (tests already passed in testing workflow)
     transitions:
-      visual: verify_visual
+      visual: load_skills_verify
       logic: done
 
+  load_skills_verify:
+    skills:
+      - coding-skill-selector
+      - ?debug-bridge
+      - ?project-skill-selector
+    transitions:
+      continue: verify_visual
+
   verify_visual:
     task: "Verify the fix visually"
     prompt: |
       Verify the original bug is fixed VISUALLY.
 
+      Tool chosen at reproduction: {{context.chosen_tool}}
+      (If empty — context was lost. Re-pick a tool now using the same approach
+      as reproduce_visual: check available MCP tools, pick the one matching the
+      platform.)
+
       MANDATORY STEPS:
-      1. Build and launch the app with debug bridge
-      2. Follow the SAME path used during reproduction
-      3. Take a screenshot and VERIFY the bug is gone
-      4. Compare before/after — describe the visual difference
+      1. Build (if applicable — native targets must be built before launch; rebuild after the fix)
+      2. Launch the app using the tool above
+      3. Follow the SAME path used during reproduction
+      4. Take a screenshot and VERIFY the bug is gone
+      5. Compare before/after — describe the visual difference
 
       DO NOT transition to `confirmed` without a screenshot proving the fix works.
     transitions:
@@ -302,7 +332,7 @@ states:
     task: "Clarify bug details"
     prompt: "Ask the user for more details about the bug."
     transitions:
-      got_info_visual: reproduce_visual
+      got_info_visual: load_skills_visual
       got_info_logic: reproduce_logic
 
   escalate:

package/templates/code-review.yaml

@@ -245,7 +245,7 @@ states:
       - No debatable concerns → `all_clear`
     transitions:
       has_concerns: discuss
-      all_clear: re_review
+      all_clear: done
 
   discuss:
     task: "Discuss debatable concerns"
@@ -269,65 +269,42 @@ states:
     transitions:
       plan: plan_changes
       code: code_changes
-      skip: re_review
+      skip: done
 
   plan_changes:
     task: "Plan agreed changes"
     sub_workflow: planning
-    on_complete: verify_functional
+    on_complete: done
     on_fail: done
 
   code_changes:
     task: "Apply agreed changes"
     sub_workflow: coding
-    on_complete: verify_functional
+    on_complete: verify_impact
     on_fail: done
 
-  verify_functional:
-    task: "Verify build and test functional changes"
-    prompt: |
-      Code was changed after discussion — verify correctness.
-
-      1. Run the project's build/compile command. Check for errors.
-         - Build fails → fix errors, rebuild until it passes.
-
-      2. Check if any agreed changes modified functionality (not just cosmetic).
-         If yes → run tests for the changed behavior. Use the project's test
-         framework, or if no automated tests cover the change, describe what
-         manual testing is needed and ask the user to verify.
-         If all changes were cosmetic → build passing is sufficient.
-
-      After verification:
-      - `pass` → proceed to re-review
-      - `fail` → changes broke the build beyond repair, skip to re-review without the failed changes
-    transitions:
-      pass: re_review
-      fail: re_review
-
-  re_review:
-    task: "Re-review changed files for cascading issues"
+  verify_impact:
+    task: "Verify build, tests, and cross-file impact"
     prompt: |
-      Previous fixes may have unblocked new issues or introduced cascading problems.
-      Re-review ALL files that were modified in previous phases (apply_fixes, code_changes).
+      Code was changed after discussion — verify correctness and impact.
 
-      1. Get the list of files changed since the review started (git diff or track from context).
-      2. If no files were changed (e.g. discussion resulted in no changes) → `clean`.
-      3. Spawn file-review agents for changed files with `scope: diff`
-         (review only the new changes, not the entire file again).
-         Use the same agent spawning pattern as `review_delegate`.
-      4. Collect reports.
+      1. Run the project's build/compile command. Fix errors and rebuild until green.
 
-      If new issues found → `has_issues` (back to summarize for another cycle).
-      If clean pass (0 new findings) → `clean`.
+      2. Run tests for changed behavior. If no automated tests cover the change,
+         describe what manual testing is needed and ask the user to verify.
+         Cosmetic-only changes → build passing is sufficient.
 
-      **Iteration limit**: check context key `re_review_count`. If >= 3, transition
-      `clean` regardless — diminishing returns after 3 cycles. Otherwise, increment
-      the counter via `context_set(key: "re_review_count", value: "<N+1>")`.
+      3. Cross-file impact check: for each file modified during discussion,
+         identify other files in the original review set that use its changed
+         APIs/symbols. Quickly inspect those call sites for compile-time or
+         obvious behavior breakage. This is NOT a full re-review — only check
+         consumers of what changed.
 
-      The goal is to loop until a clean pass — fixes stabilize and no new issues appear.
+      After verification, transition `complete`. The coding sub-workflow already
+      reviewed the files it modified. Planning produces a plan only — review
+      happens later when the plan is executed via the coding workflow.
     transitions:
-      has_issues: summarize
-      clean: done
+      complete: done
 
   done:
     prompt: "Review complete."

package/templates/file-review.yaml

@@ -26,6 +26,30 @@ states:
          Check if similar code already exists elsewhere that could be reused or unified.
       4. Understand what changed and why.
 
+      Then → transition `done`.
+    transitions:
+      done: determine_authorship
+
+  determine_authorship:
+    task: "Determine code authorship"
+    prompt: |
+      Decide whether this file is user-authored or external/forked/vendored —
+      this gates which personal rules apply downstream (style + architecture).
+
+      Strong signals it is external / forked / vendored (set "false"):
+      - `git remote -v` points to a third-party upstream the user does not control.
+      - README/CLAUDE.md mentions "fork of X", "based on X", "vendored from X".
+      - Path lives under `node_modules/`, `vendor/`, `third_party/`, a haxelib
+        cache, or similar dependency location.
+
+      Save the verdict:
+      - ANY strong external signal present → `context_set({ key: "is_user_authored", value: "false" })`
+      - Otherwise → `context_set({ key: "is_user_authored", value: "true" })`
+
+      Default to "true" when signals are ambiguous — external markers are reliably
+      detectable; user-authored markers are not, so ambiguity falls on the
+      personal-rules side. Do not block on user input.
+
       Then → transition `done`.
     transitions:
       done: build_checklist
@@ -33,7 +57,14 @@ states:
   build_checklist:
     task: "Build style checklist from skills"
     prompt: |
-      Build a concrete style checklist from loaded skills:
+      Check `{{context.is_user_authored}}`. Treat empty as `"true"` (default).
+
+      If `"false"` → personal style rules do not apply. Output:
+      "Skipping personal-style checklist — file is external/forked/vendored.
+      check_style will use only language-agnostic checks."
+      Then → transition `done`.
+
+      Otherwise (user-authored), build a concrete style checklist from loaded skills:
 
       Process each loaded `lang-*` and `preferences-*` skill ONE AT A TIME.
       For each skill:
@@ -123,27 +154,46 @@ states:
   check_style:
     task: "Check code style"
     prompt: |
+      Authorship: `{{context.is_user_authored}}` (set in determine_authorship).
+      Treat empty as `"true"` (default).
+
+      If `"false"` (external / forked / vendored):
+      Skip the personal-style checklist entirely. Match THAT codebase's existing
+      style — a one-line patch into a foreign project should look like it belongs.
+      Only flag issues from the language-agnostic checks below.
+
+      Otherwise (user-authored):
       Go through EVERY item in your style checklist from build_checklist.
+      Personal rules are authoritative — a neighboring file violating the same
+      rule is a bug replicated in both places, not a valid pattern. Flag it.
+      Never dismiss a checklist hit with "the sibling does it this way".
+
       Scope: `diff` = scan only lines in the diff; `full` = scan entire file.
       If a line appears in the diff, it is part of the change and must meet
       current style standards. Report with file:line.
 
-      Additionally check:
+      Language-agnostic checks (apply regardless of authorship):
       - [ ] Missing type annotations: if a variable/field/parameter has no explicit type
             and adding one would improve clarity, recommend it — but ONLY after you find
             the actual type definition in the codebase (via Grep/Read). Never guess types.
             Use short type names + import, not fully qualified paths (e.g. `MyClass`,
             not `com.package.MyClass`) — unless a name conflict or conditional compilation
             requires the full path (check loaded lang/preferences skills for specifics).
-      - [ ] Naming: consistent with conventions in neighboring files
+      - [ ] Naming: consistent with the conventions of THIS codebase (the user's own
+            for user-authored, the upstream's for external).
       - [ ] Member ordering: new fields and methods must respect class member order
             (constants → public vars → private vars → constructor → instance methods →
             static methods). Check that added code is in the right section, not just
             appended next to related code.
       - [ ] Duplication: does this duplicate existing code? Search the codebase.
-      - [ ] Dead code: unused imports, commented-out code (dead guards checked in check_correctness)
+      - [ ] Dead code: unused imports, commented-out code (dead guards checked in check_correctness).
+            Imports used only inside conditional-compilation blocks are dead in
+            the other branches — reference such symbols via fully qualified name
+            inside the conditional block instead of adding a top-level import.
       - [ ] Complexity: overly nested logic, functions doing too many things
-      - [ ] Consistency: does new code follow patterns in the same module?
+      - [ ] Consistency: does new code follow patterns in the same module of THIS
+            codebase? (For user-authored code, this means the user's own conventions —
+            not a license to replicate violations from sibling files.)
 
       Then → transition `done`.
     transitions:
@@ -152,17 +202,24 @@ states:
   check_architecture:
     task: "Check architecture"
     prompt: |
-      Check structural quality (scope: `diff` = changed code, `full` = entire file):
+      Authorship: `{{context.is_user_authored}}` (set in determine_authorship).
+      Treat empty as `"true"` (default).
+
+      Check structural quality (scope: `diff` = changed code, `full` = entire file).
+
+      Language-agnostic checks (apply regardless of authorship):
       - [ ] Unnecessary indirection: wrapper classes/methods that just forward calls
       - [ ] Dead abstractions: interface with one implementation, base class with one subclass
       - [ ] God class: file doing too many unrelated things — should it be split?
       - [ ] Overcomplicated: deep inheritance or nesting that could be flattened
       - [ ] Wrong level: logic placed in the wrong layer (UI doing data work, model doing display)
       - [ ] Stale patterns: old patterns that predate current codebase conventions
+      - [ ] Complexity without pain: abstractions, patterns, or layers added for hypothetical future needs
+
+      Architecture-skill checks (skip ONLY when `is_user_authored` is `"false"`):
       - [ ] Initialization in parent: parent manually triggers component's internal setup — should be inside the component
       - [ ] Serialization in runtime state: serialize/deserialize mixed into state classes — should be separate
       - [ ] Event re-dispatch duplication: re-dispatched event carries fields that duplicate event.target
-      - [ ] Complexity without pain: abstractions, patterns, or layers added for hypothetical future needs
 
       Only flag real simplification opportunities — don't invent hypothetical improvements.
 

package/templates/planning.yaml

@@ -56,6 +56,10 @@ states:
       running automatically in its review/testing states. Including them causes
       the agent to run manual builds that conflict with the automated pipeline.
 
+      **Brevity**: fewer steps. Skip what the user can infer from the task. Don't
+      restate the task back at the top of the plan. No filler sections (`Overview`,
+      `Background`) unless the task is genuinely novel and context is needed.
+
       **Steps in plan mode:**
       1. Explore codebase (Read/Glob/Grep or Plan/Explore subagents)
       2. Design approach

package/templates/skills/architecture/SKILL.md

@@ -43,6 +43,17 @@ When a component needs setup logic (creating initial children, dispatching init
 
 **Why:** Parent shouldn't know component's internal structure. If internals change, only the component changes — parent stays untouched.
 
+## Runtime State Belongs to the Element Owner
+
+When a UI element's state depends on app data (enabled/disabled, tooltip, label), the **decision logic** belongs in the component that owns the element — not in a parent coordinator.
+
+**DON'T:** Parent computes `blocked = ...`, then sets `child.button.disabled = blocked` and `tooltipManager.add(child.button, text)`
+**DO:** Parent passes raw data (`updateState(hasX, count)`), the owner decides and applies
+
+**Why:** The owner already manages the element's other state (tooltips on language change, default values). If the parent bypasses the owner, state changes from different sources (language change, limit check) overwrite each other. When the owner controls all paths, it can cache state and re-apply consistently.
+
+**Symptom of violation:** a parent sets a custom tooltip on a child's element, then a language-change handler in the child overwrites it — because the child doesn't know about the parent's custom state.
+
 ## Serialization Doesn't Belong in Runtime State
 
 **DON'T:** Put `serialize()`/`deserialize()` methods directly in runtime/state classes
@@ -52,4 +63,70 @@ When a component needs setup logic (creating initial children, dispatching init
 
 **GOTCHA at review:** Inline serialization "works" and passes tests — the violation is structural, not functional. Actively check for it: if a class has both state logic AND `serialize`/`deserialize`, flag it.
 
+## Event Re-dispatch: Don't Duplicate `target`
+
+When a composite re-dispatches events from children, `event.target` already identifies the original source.
+
+**DON'T:** Add a `targetSource` / `originalSender` field to carry the child reference through re-dispatch
+**DO:** Only add fields for data the child genuinely doesn't know (e.g. its index in the parent's collection)
+
+**Rule:** Every field in an event must have at least one consumer. "Might be useful later" fields are overengineering.
+
+## Trace Full Data Flow Before Designing
+
+Before proposing ANY storage or data mechanism, trace the COMPLETE path:
+1. **Where is it written?** (file save, cache build, sync download)
+2. **How is it serialized?** (binary format, XML, DB column)
+3. **How is it loaded?** (deserialized, copied, transformed)
+4. **How does it reach the consumer?** (direct reference, copy, event)
+
+**DON'T:** Propose new storage mechanisms (counters, file scanning, new DB columns) before checking if the project already has infrastructure for this type of data.
+**DO:** Find how analogous fields in the same system are stored and propagated. Follow the same path.
+
+Common mistake: understanding WHERE data is stored but not HOW it flows through intermediate layers (serialization, copy-on-read, event dispatch). Missing one copy step = silent data loss.
+
 When in doubt: **suggest the boring, proven, simple solution.**
+
+## Guard Externalization: Clean Up the Callee
+
+When moving a conditional guard from inside a function to its call site (wrapping the call in `if`), immediately remove the now-redundant parameter/branch from the callee.
+
+**DON'T:** Make the call conditional but leave the parameter that's now always the same value
+**DO:** Remove the parameter entirely — the caller's `if` IS the guard now
+
+**Why:** A parameter that's always `true` (or always `false`) is dead code. It misleads readers into thinking there's a second code path. Finish the refactor in one step.
+
+## Refactoring Toward Shared State: Preserve Semantic Categories
+
+When moving filtering logic from a caller into a shared state manager (allocator, registry, cache), verify the manager's API distinguishes all semantic categories the original code handled.
+
+**DON'T:** Replace an explicit filter with a generic state query that collapses distinct categories
+**DO:** List every category the original filter distinguished, confirm the API preserves each — or extend the API
+
+**Example:** A resource allocator tracks "taken by owner X". Original code built a `forbidden` set from only *manually* assigned resources — auto-assigned ones were excluded because reallocation would reassign them. Replacing `forbidden` with `allocator.isAvailable()` collapsed manual and auto into one check, blocking resources that should have been selectable.
+
+**Rule:** A generic `isX()` that returns the same answer for semantically different states is a lossy abstraction. Before using it, check: does the original code care *why* the state is set, not just *that* it's set?
+
+## Refactoring Scope: Don't Escalate Lifecycle
+
+When refactoring an inline operation into a two-phase approach (set intent + execute), match the original operation's lifecycle scope.
+
+**DON'T:** Wrap the new orchestration in container-level stop/start if the original operated at element level
+**DO:** Let existing element-level lifecycle (setters, listeners) handle transitions — they already work
+
+**Example:** A self-contained `selectNext()` swapped one item inline (release + take + index setter handled stop/start). Refactored to set intent + reallocate(). Added container-level stop/start around it. But start had an `isAvailable()` gate requiring a minimum active count — a constraint the original inline operation never hit. The element-level index setter already stops the old item and starts the new one.
+
+**Rule:** Before adding lifecycle management to a refactored operation, check: did the original operation need it? If the original worked within existing infrastructure (setters, listeners), the refactored version should too.
+
+## New Canonical Path → Ask About Legacy Parallels
+
+When you add a new canonical input/output mechanism that does the same thing as existing legacy paths (e.g. UI buttons that replicate keyboard shortcuts, a typed API replacing an untyped one, a structured event replacing a string topic), the default urge is to keep the legacy paths "for compatibility" and build a coexistence shim. Don't.
+
+**DON'T:** Silently keep legacy paths AND add a coexistence shim (coordinate gates, target-checks, debouncers) so the new and old don't fight.
+**DO:** Surface the question explicitly: "should we remove the old paths, or keep them?" — and prefer single-source by default.
+
+**Example:** Added UI panel buttons that already handle the full action flow. Kept the original canvas tap (= fire) AND keyboard shortcuts (= fire). Adding the buttons made every panel click double-fire (tap dispatched on both panel and canvas), so a Y-coord gate was introduced to suppress canvas handling over the panel area. After confirming "remove keys and canvas-click entirely, the buttons are the way" — the gate became dead code, deleted in the same pass.
+
+**Symptom of violation:** you find yourself adding gating logic (coordinate checks, `if (ev.target === ...)` filters, `inflight` flags) just to make two input paths coexist when one is meant to subsume the other.
+
+**Rule:** A new canonical mechanism is an opportunity to remove the old one, not to add a referee between them. If the user didn't explicitly ask to keep both, ask before keeping both.

package/templates/skills/aws-lambda/SKILL.md

@@ -0,0 +1,44 @@
+---
+name: aws-lambda
+description: AWS Lambda .NET deployment gotchas
+---
+
+# AWS Lambda + .NET Gotchas
+
+## VPC Networking
+
+- **Subnet IP exhaustion on scale-up.** Each ENI consumes an IP from subnet. ENI account limit = 350 default. New instances silently fail to start when exhausted.
+- **ENI creation lag: up to 90s** when function is created or VPC config changes. Invocations during this window get unpredictable cold starts.
+- **S3/Secrets Manager calls timeout silently (not connection refused)** when Lambda is in VPC without NAT/VPC Endpoint. Easy to misdiagnose as SDK bug.
+
+## Logging
+
+- **`Console.Error.WriteLine()` does NOT appear in CloudWatch.** Use `ILambdaContext.Logger.LogLine()` in handler code or `ILogger` in ASP.NET pipeline.
+- **Missing `logs:CreateLogGroup` permission = logs silently lost.** Log group created lazily on first invoke. No permission = no log group = nowhere to write = no error visible anywhere.
+
+## ASP.NET Core in Lambda
+
+- **`UseHttpsRedirection()` hangs Lambda.** No Kestrel HTTPS port. Middleware hangs trying to discover one (aws-lambda-dotnet #1543). Only enable in Development.
+- **Response > 6MB crashes INIT phase, not 413.** `APIGatewayHttpApiV2ProxyFunction` throws exception in Lambda runtime client during initialization, taking down the entire ASP.NET Core app instead of returning HTTP error.
+
+## S3 from Lambda
+
+- **SDK default timeout = infinite, retry = 4 with exponential backoff.** Total wait 30-60s can exceed Lambda timeout. Always set `Timeout` and `MaxErrorRetry` on `AmazonS3Config`.
+- **`AmazonS3Client` without explicit region defaults to `us-east-1`.** Cross-region requests are slower. No error if IAM allows it — silent performance degradation.
+- **IMDS credential error is cached.** If Instance Metadata Service is transiently unavailable during first init, SDK caches the error. All subsequent requests fail until container restart.
+
+## .NET Cold Start
+
+- **ReadyToRun (R2R) compiled on macOS/Windows is ignored in Lambda (Amazon Linux 2).** Must publish with `--runtime linux-x64`. R2R flag silently has no effect cross-platform.
+- **Container deploy: INIT phase is billed. ZIP deploy: INIT phase is free.** For .NET with heavy startup, significant cost difference.
+- **EF Core `Database.Migrate()` default command timeout = 30s.** Heavy migrations fail with EF timeout, not Lambda timeout. Set `CommandTimeout` explicitly.
+
+## Function URL
+
+- **CORS headers not returned without `Origin` header in request.** Preflight without Origin returns 200 but no Access-Control-* headers. Browser blocks the actual request. Hard to diagnose — server returns 200.
+
+## IAM & Deployment
+
+- **`UpdateFunctionCode` + `UpdateFunctionConfiguration` cannot run simultaneously.** Gives `ResourceConflictException`. CI/CD must wait for `LastUpdateStatus: Successful` between calls.
+- **`State: Active` does NOT mean ready.** `LastUpdateStatus: InProgress` means API calls will fail with `ResourceConflictException`. Must check both fields.
+- **Provisioned Concurrency on alias/version doesn't help `$LATEST`.** Must invoke the aliased version to benefit. CLI deploys without explicit alias still get cold starts.