Skip to content

Feat/exec control flow tools - #2165

Closed
MiMoHardFather wants to merge 2 commits into
mainfrom
feat/exec-control-flow-tools
Closed

Feat/exec control flow tools#2165
MiMoHardFather wants to merge 2 commits into
mainfrom
feat/exec-control-flow-tools

Conversation

@MiMoHardFather

Copy link
Copy Markdown
Collaborator

Issue / context (if applicable)

Type of change

Bug fix / New feature / Refactor / Documentation — keep the ones that apply and delete the rest.

What does this PR do?

Briefly describe the problem, what changed, and why this approach works.

How did you verify your code works?

What did you test, and how can a reviewer reproduce the result?

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

- Replace dual edit/sed patches with apply_patch only in tool-script GPT reference
- Add comprehensive PROMPT.md sections 6-10 for apply_patch workflow, git diffs,
  common failures, format examples, and structured verification flow
- Enforce GPT: test via bash; Claude/Exec: use verify skill
- Add test coverage for registry GPT script and structured verification prompt
- Update all test expectations for unified tool-script reference
- Clean up whitespace and tighten wording in bash tools and prompts
…ntly

fix: use apply_patch consistently and add structured verify flow
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