taskchef 7.9.0 → 7.9.1

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.9.0",
+  "version": "7.9.1",
   "description": "Dispatch work from a data-only workspace to visible Codex project tasks.",
   "author": {
     "name": "Favo Yang",

package/README.md

@@ -104,7 +104,6 @@ For example, TaskChef generates this shape:
 
 ```text
 Fix duplicate charges after a retry and add a regression test.
-
 <!-- taskchef_id=c0f010ff-84f2-4838-a69d-0ff1f5d721d7 -->
 Use $taskchef-executor to execute and report this delegated TaskChef assignment.
 ```

package/docs/spec.md

@@ -116,10 +116,10 @@ the final link, preserving the delegate skill's immediate-return contract.
 1. The dispatcher MUST call `prepare_dispatch` once per outcome.
 2. It MUST choose exactly one configured project and exact native-project path.
 3. It MUST build the instruction with the user's outcome beginning on line 1
-   and remaining uninterrupted, followed by one blank line, the returned
-   marker, and exactly one concise explicit `$taskchef-executor` invocation on
-   the final line. It MUST NOT inline the executor protocol into a new
-   instruction.
+   and remaining uninterrupted, followed by exactly one newline, the returned
+   marker, one newline, and exactly one concise explicit `$taskchef-executor`
+   invocation on the final line. It MUST NOT place a blank line before or after
+   the marker or inline the executor protocol into a new instruction.
 4. It MUST call `record_task` with `threadId: null` before native creation.
 5. It MUST create exactly one native Codex executor and return immediately.
 6. The executor MUST read its own `CODEX_THREAD_ID` and call `link_task`
@@ -136,9 +136,10 @@ If native creation fails after recording, the dispatcher MUST call
 A link failure MUST remain visible and retryable; the executor MUST report it
 visibly and MUST NOT continue substantive work.
 
-Previously recorded instructions with a first-line HTML marker, the older
-first-line `# taskchef_id=<full UUID>` heading, or the former blank line and
-inline executor protocol MUST remain marker-readable and executable. Their
+Previously recorded instructions with the former blank line before a trailing
+marker, a first-line HTML marker, the older first-line
+`# taskchef_id=<full UUID>` heading, or the former blank line and inline
+executor protocol MUST remain marker-readable and executable. Their
 `report_result` calls MUST remain supported by the deprecated alias. New
 instructions MUST use the trailing marker and explicit executor skill contract
 above. A historical first-line instruction with an executor-skill invocation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.9.0",
+  "version": "7.9.1",
   "description": "A non-blocking interactive dispatcher for visible Codex tasks.",
   "license": "MIT",
   "author": "Favo Yang",

package/skills/taskchef-delegate/SKILL.md

@@ -52,8 +52,9 @@ stop and report that the TaskChef plugin must be reloaded or installed.
 
    - Begin with the actual assignment on the first line and keep its complete
      body uninterrupted.
-   - After the assignment, add one blank line and the preparation's exact
-     marker on its own line.
+   - After the assignment's final character, add exactly one newline and the
+     preparation's exact marker on its own line. Do not add a blank line before
+     or after the marker.
    - Immediately after the marker, end the instruction with exactly:
      `Use $taskchef-executor to execute and report this delegated TaskChef assignment.`
    - Include exactly one marker and exactly one executor-skill invocation.

package/skills/taskchef-executor/SKILL.md

@@ -9,12 +9,13 @@ Own and execute the delegated assignment in the current Codex task. Do not
 re-dispatch it merely because it concerns TaskChef or a configured project.
 Explicit requests to delegate separate work remain valid.
 
-New instructions present the complete assignment first, followed by one blank
-line, the exact `<!-- taskchef_id=<full UUID> -->` marker, and the final
-explicit skill invocation. Treat that UUID as the TaskChef task ID. The
-assignment is everything before the blank line that precedes the marker; the
-marker and invocation are lifecycle scaffolding, not part of the deliverable.
-Require exactly one marker and do not infer an ID from similar prose.
+New instructions present the complete assignment first, followed by exactly one
+newline, the exact `<!-- taskchef_id=<full UUID> -->` marker, one newline, and
+the final explicit skill invocation. There is no blank line before or after the
+marker. Treat that UUID as the TaskChef task ID. The assignment is everything
+before the marker; the marker and invocation are lifecycle scaffolding, not
+part of the deliverable. Require exactly one marker and do not infer an ID from
+similar prose.
 
 ## Start every execution turn
 
@@ -69,12 +70,13 @@ Existing delegated tasks may include the former inline ownership, linking, and
 re-dispatching. Prefer `report_state` when available. If an older installed
 TaskChef exposes only `report_result`, follow its inline protocol; after an
 upgrade, the deprecated `report_result` alias remains available for exact
-legacy retries. Also accept historical instructions whose exact HTML marker is
-the first line, with or without the former blank line, and the older exact
-first-line `# taskchef_id=<full UUID>` heading. These compatibility forms do
-not change the identity or lifecycle rules above. For either first-line form,
-the assignment follows the marker. Ignore the final executor invocation and
-any recognizable former inline ownership, linking, working-state, or
+legacy retries. Also accept historical instructions with the former blank line
+before a trailing marker, an exact HTML marker on the first line with or
+without the former blank line, or the older exact
+first-line `# taskchef_id=<full UUID>` heading. These compatibility forms do not
+change the identity or lifecycle rules above. For either first-line form, the
+assignment follows the marker. Ignore the final executor invocation and any
+recognizable former inline ownership, linking, working-state, or
 result-reporting paragraphs as lifecycle scaffolding; execute the remaining
 task-specific body. Require non-whitespace task-specific content and never
 treat an invocation by itself as an assignment.

package/src/delegation.js

@@ -162,14 +162,18 @@ export function parseTaskChefMarker(instruction) {
     return hasHistoricalAssignment() ? id : null;
   }
   const executorSkillReferences = instruction.match(/\$taskchef-executor\b/gi) ?? [];
+  const hasCompactBoundary = index >= 1
+    && lines.at(index - 1).trim().length > 0;
+  const hasHistoricalBlankBoundary = index >= 2
+    && lines.at(index - 1) === ""
+    && lines.at(index - 2).trim().length > 0;
+  const assignmentEnd = hasCompactBoundary ? index : index - 1;
   const isTrailingScaffold = index === lines.length - 2
-    && index >= 2
+    && (hasCompactBoundary || hasHistoricalBlankBoundary)
     && lines[0].trim().length > 0
-    && lines.at(-3) === ""
-    && lines.at(-4).trim().length > 0
     && lines.at(-1) === EXECUTOR_SKILL_INVOCATION
-    && hasTaskSpecificContent(lines.slice(0, index - 1))
-    && !lines.slice(0, index - 1).some((line) => HISTORICAL_EXECUTOR_SCAFFOLD_LINES.has(line))
+    && hasTaskSpecificContent(lines.slice(0, assignmentEnd))
+    && !lines.slice(0, assignmentEnd).some((line) => HISTORICAL_EXECUTOR_SCAFFOLD_LINES.has(line))
     && executorSkillReferences.length === 1;
   return isTrailingScaffold ? id : null;
 }
@@ -198,7 +202,7 @@ export function prepareDelegation(instruction, { taskId = randomUUID() } = {}) {
   const id = requireUuid(taskId);
   return {
     id,
-    instruction: `${body}\n\n${taskChefMarker(id)}\n${EXECUTOR_SKILL_INVOCATION}`,
+    instruction: `${body}\n${taskChefMarker(id)}\n${EXECUTOR_SKILL_INVOCATION}`,
   };
 }