Skip to content

feat(ACP): integrate ACP permission handling into session prompt#3336

Merged
rekram1-node merged 3 commits into
anomalyco:cleanup-acpfrom
yetone:feature/acp-permission-handling
Oct 24, 2025
Merged

feat(ACP): integrate ACP permission handling into session prompt#3336
rekram1-node merged 3 commits into
anomalyco:cleanup-acpfrom
yetone:feature/acp-permission-handling

Conversation

@yetone

@yetone yetone commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

Summary

Testing

  • bun turbo typecheck

@yetone yetone changed the title Integrate ACP permission handling into session prompt feat(ACP): integrate ACP permission handling into session prompt Oct 21, 2025
@yetone yetone force-pushed the feature/acp-permission-handling branch from 4d79cd4 to c18bdc4 Compare October 21, 2025 18:40
Comment thread packages/opencode/src/session/prompt.ts
Comment thread packages/opencode/src/session/prompt.ts Outdated
@yetone yetone force-pushed the feature/acp-permission-handling branch from 566aa8d to 368b73b Compare October 22, 2025 08:42
@rekram1-node

Copy link
Copy Markdown
Collaborator

ill take a look at this today, thanks!

@rekram1-node

rekram1-node commented Oct 23, 2025

Copy link
Copy Markdown
Collaborator

merge conflict now :/

I think I am gonna try to reorganize the acp stuff I dont want it to be so coupled to the core logic if we can help it

@yetone

yetone commented Oct 23, 2025

Copy link
Copy Markdown
Contributor Author

@rekram1-node Do you tend to refactor ACP in this PR or create a new PR after this one is merged?

@rekram1-node

Copy link
Copy Markdown
Collaborator

if you resolve the merge conflicts I will merge the pr, either to a refactor branch or directly but your name will be on the changes at the end of it all either way :)

@yetone yetone force-pushed the feature/acp-permission-handling branch from 368b73b to dca9653 Compare October 23, 2025 10:25
@yetone

yetone commented Oct 23, 2025

Copy link
Copy Markdown
Contributor Author

Code conflict resolved.

@rekram1-node rekram1-node changed the base branch from dev to cleanup-acp October 24, 2025 05:28
@rekram1-node rekram1-node merged commit 9ef0008 into anomalyco:cleanup-acp Oct 24, 2025
@rekram1-node

Copy link
Copy Markdown
Collaborator

merged into my cleanup branch, will prolly have something cleaned up and shipped tmr

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.

[ACP] Tools that have ask permision hang when invoked

3 participants