taskchef 7.12.1 → 7.12.2

package/.codex-plugin/plugin.json

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

package/README.md

@@ -88,9 +88,9 @@ $taskchef-delegate In payments, add structured logs for failed retries and test
 
 TaskChef prepares a UUID and marker, persists the task before native creation,
 creates the executor, and returns its task link. New executor instructions keep
-the assignment visible from the first line, then place an explicit
-`$taskchef-executor` invocation immediately before the final correlation
-marker. That skill reads the executor's own `CODEX_THREAD_ID`, self-links, and
+the assignment visible from the first line, leave one blank line, then place
+an explicit `$taskchef-executor` invocation immediately before the final
+correlation marker. That skill reads the executor's own `CODEX_THREAD_ID`, self-links, and
 reports lifecycle state. Independent outcomes may become separate executors;
 dependent work should stay together.
 
@@ -104,6 +104,7 @@ For example, TaskChef generates this shape:
 
 ```text
 Fix duplicate charges after a retry and add a regression test.
+
 Use $taskchef-executor to execute and report this delegated TaskChef assignment.
 <!-- taskchef_id=c0f010ff-84f2-4838-a69d-0ff1f5d721d7 -->
 ```

package/docs/spec.md

@@ -119,10 +119,11 @@ 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 exactly one newline, exactly one
-   concise explicit `$taskchef-executor` invocation, one newline, and the
-   returned marker on the final line. It MUST NOT place blank lines around the
-   invocation or marker or inline the executor protocol into a new instruction.
+   and remaining uninterrupted, followed by exactly two newline characters
+   (one blank line), exactly one concise explicit `$taskchef-executor`
+   invocation, one newline, and the returned marker on the final line. It MUST
+   NOT place a blank line between the invocation and 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`
@@ -143,6 +144,7 @@ visibly and MUST NOT continue substantive work.
 
 Previously recorded instructions with the trailing marker before the
 invocation, with or without the former blank line before that marker; a
+former compact assignment-to-invocation boundary with the marker last; 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

package/docs/workflows.md

@@ -107,8 +107,9 @@ Record-before-create makes native creation failure observable. Executor
 self-linking removes dispatcher-side polling, task search, title matching, and
 parent/child identity inference.
 
-The generated task begins with the complete assignment, then places one explicit
-`$taskchef-executor` invocation immediately before its final marker. Older
+The generated task begins with the complete assignment, leaves one blank line,
+then places one explicit `$taskchef-executor` invocation immediately before its
+final marker. Older
 recorded tasks with first-line HTML or heading markers and former inline
 protocol remain readable; the deprecated `report_result` alias preserves their
 semantic callbacks.

package/package.json

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

package/skills/taskchef-delegate/SKILL.md

@@ -53,12 +53,12 @@ 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's final character, add exactly one newline and this
-     invocation on its own line:
+   - After the assignment's final character, add exactly two newline characters
+     so there is one blank line before this invocation on its own line:
      `Use $taskchef-executor to execute and report this delegated TaskChef assignment.`
    - Immediately after the invocation, end the instruction with one newline
-     and the preparation's exact marker on its own final line. Do not add blank
-     lines around the invocation or marker.
+     and the preparation's exact marker on its own final line. Do not add a
+     blank line between the invocation and marker.
    - Include exactly one marker and exactly one executor-skill invocation.
    - Do not inline executor ownership, identity, linking, or result-reporting
      protocol. The explicitly invoked executor skill owns those mechanics.

package/skills/taskchef-executor/SKILL.md

@@ -9,10 +9,11 @@ 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 exactly one
-newline, the explicit skill invocation, one newline, and the exact
-`<!-- taskchef_id=<full UUID> -->` marker on the final line. There are no blank
-lines around the invocation or marker. Treat that UUID as the TaskChef task ID.
+New instructions present the complete assignment first, followed by exactly two
+newline characters (one blank line), the explicit skill invocation, one
+newline, and the exact `<!-- taskchef_id=<full UUID> -->` marker on the final
+line. There is no blank line between the invocation and marker. Treat that UUID
+as the TaskChef task ID.
 The assignment is everything before the invocation; the invocation and marker
 are lifecycle scaffolding, not part of the deliverable. Require exactly one
 marker and do not infer an ID from similar prose.
@@ -79,7 +80,8 @@ 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 trailing instructions that place the
 marker before the invocation, with or without the former blank line before the
-marker; an exact HTML marker on the first line with or without the former blank
+marker; the former compact assignment-to-invocation boundary with the marker
+last; 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

package/src/delegation.js

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