Repository navigation
Conversation
|
Friendly follow-up: this PR is intentionally limited to merging session permissions into the legacy |
39eaa86 to
f6ac9ed
Compare
f6ac9ed to
c3ee827
Compare
|
Hi @rekram1-node, since #48106 is assigned to you, could you approve the fork workflows for this PR and take a look when you have time? I've rebased onto the latest The change remains limited to merging session permissions into the Task description. Thank you! |
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
|
Hi @rekram1-node, this fix is still actively maintained. Could you reopen this PR after the automated cleanup and approve the fork CI workflows? The branch was rebased onto I tried reopening the PR directly, but GitHub rejected the request. Thank you for taking a look. |
Issue for this PR
Closes #48106
Type of change
What does this PR do?
Pass merged agent and session permissions to
describeTask, so its available-agent list reflects session deny, allow, and ask overrides, matching execution-time evaluation.This resubmits only the description permission merge from #41100, which was closed by automated cleanup. No changes to target visibility policy or execution checks.
How did you verify your code works?
Using Bun 1.3.14 in
packages/opencode:bun test test/tool/registry.test.ts test/session/tools.test.ts test/permission-task.test.ts --timeout 30000— 43 pass.bun typecheck— pass; the pre-push full-workspace typecheck also passed (30/30 tasks).git diff --checkpass.Screenshots / recordings
No UI changes.
Checklist