Skip to content

feat(tools): onParameterParseFailure teardown boundary (split 4/6 of #1066) - #67

Closed
easonLiangWorldedtech wants to merge 1 commit into
p1066/u5-streaming-failure-capturefrom
p1066/u3-parse-failure-boundary
Closed

easonLiangWorldedtech wants to merge 1 commit into
p1066/u5-streaming-failure-capturefrom
p1066/u3-parse-failure-boundary

Conversation

@easonLiangWorldedtech

Copy link
Copy Markdown
Owner

U3 — onParameterParseFailure teardown boundary

Part of the PR Zoo-Code-Org#1066 split (tracking issue Zoo-Code-Org#703) split (content source of record: 72143527fd33306e5541116093c2cbf803cce9e0..pr-1066-audit).

Why this unit exists: BaseTool.onParameterParseFailure() teardown boundary plus the WriteToFileTool override, so a truncated final block still finalizes the open partial ask.

Boundaries

  • base: feedc5dafbf4
  • head: e2a03d949257
  • content source: 72143527fd33306e5541116093c2cbf803cce9e0..pr-1066-audit (local)

Fidelity (machine-verified)

zdt split verify --contract U3.json --worktree <wt> --head e2a03d949257

Result: PASS — standalone 253 a+d / 3 files (UNDER-SOFT)

  • src/core/tools/BaseTool.ts: OK (content subset of source)
  • src/core/tools/WriteToFileTool.ts: OK (content subset of source)
  • src/core/tools/__tests__/writeToFileTool.spec.ts: OK (content subset of source)

Design contract

Verification (this unit, as pushed)

  • Tests: 243 passed / 5 skipped
  • changed-line coverage: 20 covered / 0 uncovered — PASS
  • ESLint: clean on every touched file; suppression counts unchanged
  • No .changeset file, no CHANGELOG edit.

Recreate policy

If the bot stalls on a pre-merge check and the existing head cannot obtain bot review/approval (empty-commit re-trigger attempted and failed), the unit is recreated from the tagged content source of record — never from a per-PR head. At most 1 PR per issue.

@easonLiangWorldedtech

Copy link
Copy Markdown
Owner Author

Moved to the upstream repo (base = main); the split chain is tracked in Zoo-Code-Org#703.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant