macp-sdk-typescript 0.10.0 → 0.11.0

package/dist/projections/task.js

@@ -2,35 +2,14 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TaskProjection = void 0;
 const constants_1 = require("../constants");
-const logging_1 = require("../logging");
-class TaskProjection {
+const base_1 = require("./base");
+class TaskProjection extends base_1.BaseProjection {
+    mode = constants_1.MODE_TASK;
     tasks = new Map();
     updates = [];
     completions = [];
     failures = [];
-    /**
-     * The session's accepted history, one envelope per unique `message_id`. See
-     * `BaseProjection.transcript` (`src/projections/base.ts`) for the full
-     * redelivery-idempotence contract (RFC-MACP-0006 §3.2); duplicated here
-     * only as a one-line pointer.
-     */
-    transcript = [];
-    /**
-     * Cardinality anomalies recorded while replaying this projection's accepted
-     * transcript. See `BaseProjection.anomalies` (`src/projections/base.ts`)
-     * for the canonical description; duplicated here only as a one-line
-     * pointer. No built-in detection populates this for Task mode.
-     */
-    anomalies = [];
     phase = 'Pending';
-    commitment;
-    /**
-     * `message_id`s already applied to this projection. See
-     * `BaseProjection.seenMessageIds` (`src/projections/base.ts`) for the full
-     * redelivery-idempotence rationale (RFC-MACP-0006 §3.2); duplicated here
-     * only as a one-line pointer.
-     */
-    seenMessageIds = new Set();
     /**
      * The **session-level** active assignment: who currently holds the session's
      * single assignee slot, and which `task_id` they took it on.
@@ -51,36 +30,8 @@ class TaskProjection {
      * the right `TaskRecord.assignee` even on a multi-task transcript.
      */
     activeAssignment;
-    /**
-     * Apply one envelope to this projection's in-process state.
-     *
-     * Input contract: **accepted-only**, caller-maintained (`Envelope` carries
-     * no acceptance marker). Canonical source: `schemas/conformance/README.md`
-     * "Notes:", RFC-MACP-0007 §5.3, RFC-MACP-0011 §5. Full rationale and failure
-     * mode are documented once, on `BaseProjection.applyEnvelope`
-     * (`src/projections/base.ts`), and duplicated here only as a one-line
-     * pointer so six independent copies of the same prose cannot drift.
-     *
-     * Redelivery idempotence: a redelivered envelope (same `message_id`) is a
-     * non-event — not appended to `transcript`, not passed to the `switch`.
-     * See `BaseProjection.applyEnvelope` for the full RFC-MACP-0006 §3.2
-     * citation; duplicated here only as a one-line pointer.
-     */
-    applyEnvelope(envelope, protoRegistry) {
-        if (envelope.mode !== constants_1.MODE_TASK)
-            return;
-        if (envelope.messageId) {
-            if (this.seenMessageIds.has(envelope.messageId)) {
-                logging_1.logger.debug('projection redelivery ignored', {
-                    messageId: envelope.messageId,
-                    mode: envelope.mode,
-                    messageType: envelope.messageType,
-                });
-                return;
-            }
-            this.seenMessageIds.add(envelope.messageId);
-        }
-        this.transcript.push(envelope);
+    /** Handle a Task-mode envelope. See `BaseProjection.applyEnvelope` for the accepted-only input contract and redelivery/rollback semantics shared by all built-in modes. */
+    applyMode(envelope, protoRegistry) {
         const payload = protoRegistry.decodeKnownPayload(envelope.mode, envelope.messageType, envelope.payload);
         switch (envelope.messageType) {
             case 'TaskRequest': {
@@ -199,33 +150,10 @@ class TaskProjection {
                 this.phase = 'Failed';
                 break;
             }
-            case 'Commitment': {
-                this.commitment = payload;
-                this.phase = 'Committed';
-                break;
-            }
             default:
                 break;
         }
     }
-    /**
-     * True once at least one `ProjectionAnomaly` has been recorded. See
-     * `BaseProjection.hasAnomalies` (`src/projections/base.ts`) for the
-     * canonical description; duplicated here only as a one-line pointer.
-     */
-    get hasAnomalies() {
-        return this.anomalies.length > 0;
-    }
-    get isCommitted() {
-        return this.commitment !== undefined;
-    }
-    get isPositiveOutcome() {
-        if (!this.commitment)
-            return undefined;
-        const val = this.commitment.outcomePositive ??
-            this.commitment.outcome_positive;
-        return val !== undefined ? Boolean(val) : true;
-    }
     getTask(taskId) {
         return this.tasks.get(taskId);
     }

package/dist/proto-registry.d.ts

@@ -10,11 +10,25 @@ export declare class ProtoRegistry {
     /**
      * Decode an `ext.multi_round.v1` `Contribute` payload, accepting both wire
      * formats: legacy JSON (`{"value":"..."}`, still replayed verbatim from
-     * pre-proto histories) and canonical protobuf (`ContributePayload`). JSON is
-     * tried first — mirroring the runtime's permanent decode order — and the two
-     * encodings are disjoint on their first byte (`{` = 0x7b vs proto field-1
-     * tag 0x0A), so a canonical proto payload never mis-parses as JSON and vice
-     * versa. Both normalize to `{ value: string }`.
+     * pre-proto histories) and canonical protobuf (`ContributePayload`). Both
+     * normalize to `{ value: string }`.
+     *
+     * Parse-then-fallback (issue #93), not a first-byte shortcut: JSON is always
+     * attempted first, and only a `JSON.parse` failure falls through to
+     * protobuf. A first-byte check (`{` = 0x7b vs proto field-1 tag 0x0A) does
+     * not generalize — e.g. legacy JSON with leading whitespace has neither byte
+     * first, and used to reach `decodeMessage` and throw. Parity with
+     * `macp-sdk-python`'s `_decode_json_first_then_proto`, which has always
+     * worked this way. Safe because a canonical proto payload's bytes are
+     * essentially never also syntactically valid JSON.
+     *
+     * An empty payload falls through the same path: `JSON.parse('')` throws, so
+     * it reaches `decodeMessage` on zero bytes, which yields `{}` (proto3
+     * defaults) rather than a decode error. This SDK's decode layer is
+     * observational, not an acceptance gate — whether an empty `Contribute`
+     * should be rejected is a runtime-acceptance question (tracked cross-repo,
+     * `contribute_acceptance.empty_payload` in `schemas/parity/contract.json`,
+     * currently `macp-runtime`-only), not something this method decides.
      */
     private decodeMultiRoundContribute;
     private lookupType;

package/dist/proto-registry.js

@@ -169,26 +169,34 @@ class ProtoRegistry {
     /**
      * Decode an `ext.multi_round.v1` `Contribute` payload, accepting both wire
      * formats: legacy JSON (`{"value":"..."}`, still replayed verbatim from
-     * pre-proto histories) and canonical protobuf (`ContributePayload`). JSON is
-     * tried first — mirroring the runtime's permanent decode order — and the two
-     * encodings are disjoint on their first byte (`{` = 0x7b vs proto field-1
-     * tag 0x0A), so a canonical proto payload never mis-parses as JSON and vice
-     * versa. Both normalize to `{ value: string }`.
+     * pre-proto histories) and canonical protobuf (`ContributePayload`). Both
+     * normalize to `{ value: string }`.
+     *
+     * Parse-then-fallback (issue #93), not a first-byte shortcut: JSON is always
+     * attempted first, and only a `JSON.parse` failure falls through to
+     * protobuf. A first-byte check (`{` = 0x7b vs proto field-1 tag 0x0A) does
+     * not generalize — e.g. legacy JSON with leading whitespace has neither byte
+     * first, and used to reach `decodeMessage` and throw. Parity with
+     * `macp-sdk-python`'s `_decode_json_first_then_proto`, which has always
+     * worked this way. Safe because a canonical proto payload's bytes are
+     * essentially never also syntactically valid JSON.
+     *
+     * An empty payload falls through the same path: `JSON.parse('')` throws, so
+     * it reaches `decodeMessage` on zero bytes, which yields `{}` (proto3
+     * defaults) rather than a decode error. This SDK's decode layer is
+     * observational, not an acceptance gate — whether an empty `Contribute`
+     * should be rejected is a runtime-acceptance question (tracked cross-repo,
+     * `contribute_acceptance.empty_payload` in `schemas/parity/contract.json`,
+     * currently `macp-runtime`-only), not something this method decides.
      */
     decodeMultiRoundContribute(payload) {
-        if (!payload.length)
+        try {
+            const parsed = JSON.parse(payload.toString('utf8'));
+            return { value: typeof parsed.value === 'string' ? parsed.value : String(parsed.value ?? '') };
+        }
+        catch {
             return this.decodeMessage(MULTI_ROUND_CONTRIBUTE, payload);
-        if (payload[0] === 0x7b) {
-            // Leading `{` — legacy JSON. Parse and normalize to { value }.
-            try {
-                const parsed = JSON.parse(payload.toString('utf8'));
-                return { value: typeof parsed.value === 'string' ? parsed.value : String(parsed.value ?? '') };
-            }
-            catch {
-                // Not valid JSON despite the leading brace — fall through to protobuf.
-            }
         }
-        return this.decodeMessage(MULTI_ROUND_CONTRIBUTE, payload);
     }
     lookupType(typeName) {
         const lookedUp = this.root.lookupType(typeName);

package/dist/version.d.ts

@@ -4,4 +4,4 @@
  * Kept in sync with package.json by release-please (see the
  * `x-release-please-version` annotation).
  */
-export declare const VERSION = "0.10.0";
+export declare const VERSION = "0.11.0";

package/dist/version.js

@@ -7,4 +7,4 @@ exports.VERSION = void 0;
  * Kept in sync with package.json by release-please (see the
  * `x-release-please-version` annotation).
  */
-exports.VERSION = '0.10.0'; // x-release-please-version
+exports.VERSION = '0.11.0'; // x-release-please-version

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "macp-sdk-typescript",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "description": "TypeScript SDK for the MACP Rust runtime",
   "license": "Apache-2.0",
   "type": "commonjs",
@@ -55,18 +55,18 @@
   "dependencies": {
     "@grpc/grpc-js": "^1.14.3",
     "@grpc/proto-loader": "^0.8.0",
-    "@multiagentcoordinationprotocol/proto": "^0.1.9",
+    "@multiagentcoordinationprotocol/proto": "^0.1.10",
     "protobufjs": "^8.6.5"
   },
   "devDependencies": {
     "@types/node": "^26.4.0",
     "@typescript-eslint/eslint-plugin": "^8.68.0",
     "@typescript-eslint/parser": "^8.68.0",
-    "@vitest/coverage-v8": "^4.1.11",
+    "@vitest/coverage-v8": "^5.0.0",
     "eslint": "^10.9.1",
     "prettier": "^3.0.0",
     "tsx": "^4.21.0",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.11"
+    "vitest": "^5.0.0"
   }
 }