Skip to content

fix(write-to-file): clean partial state on missing-param and rooignore denial (split 6/6 of #1066) - #69

Closed
easonLiangWorldedtech wants to merge 1 commit into
p1066/u6-execute-error-path-cleanupfrom
p1066/u7-early-return-denial-cleanup
Closed

easonLiangWorldedtech wants to merge 1 commit into
p1066/u6-execute-error-path-cleanupfrom
p1066/u7-early-return-denial-cleanup

Conversation

@easonLiangWorldedtech

Copy link
Copy Markdown
Owner

U7 — early-return / rooignore-denial cleanup

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: Early-return and rooignore-denial branches clear the per-task partial state and finalize the open ask.

Boundaries

  • base: 03bde6705ef6
  • head: 4c23078d8b8c
  • content source: 72143527fd33306e5541116093c2cbf803cce9e0..pr-1066-audit (local)

Fidelity (machine-verified)

zdt split verify --contract U7.json --worktree <wt> --head 4c23078d8b8c

Result: PASS — standalone 241 a+d / 2 files (UNDER-SOFT)

  • 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: 264 passed / 5 skipped
  • changed-line coverage: 12 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