fix(agents): add PR diff and file list permissions to implementation-worker #8247

Merged
HAL9000 merged 3 commits from fix/agents/impl-worker-pr-diff-perms-8175 into master 2026-04-25 04:20:32 +00:00

3 Commits

Author SHA1 Message Date
HAL9000 a47e88016f fix(agents): add PR diff and file list permissions to implementation-worker
CI / benchmark-publish (pull_request) Has been skipped
CI / push-validation (pull_request) Successful in 47s
CI / helm (pull_request) Successful in 57s
CI / build (pull_request) Successful in 1m42s
CI / lint (pull_request) Successful in 2m11s
CI / quality (pull_request) Successful in 2m25s
CI / typecheck (pull_request) Successful in 2m30s
CI / security (pull_request) Successful in 2m30s
CI / e2e_tests (pull_request) Successful in 6m23s
CI / integration_tests (pull_request) Successful in 7m38s
CI / unit_tests (pull_request) Successful in 10m5s
CI / docker (pull_request) Successful in 1m35s
CI / coverage (pull_request) Successful in 14m40s
CI / benchmark-regression (push) Has been skipped
CI / status-check (pull_request) Successful in 3s
CI / helm (push) Successful in 31s
CI / benchmark-publish (push) Has started running
CI / build (push) Successful in 53s
CI / lint (push) Successful in 1m11s
CI / typecheck (push) Successful in 1m21s
CI / push-validation (push) Successful in 35s
CI / quality (push) Successful in 1m21s
CI / security (push) Successful in 1m31s
CI / e2e_tests (push) Successful in 4m7s
CI / integration_tests (push) Successful in 6m11s
CI / unit_tests (push) Successful in 6m37s
CI / docker (push) Successful in 1m42s
CI / coverage (push) Successful in 15m5s
CI / status-check (push) Waiting to run
CI / benchmark-regression (pull_request) Successful in 59m41s
The implementation-worker has a PR fix mode that needs to read PR changes
to understand what is failing. However, it lacked forgejo_list_pull_request_files
and forgejo_get_pull_request_diff permissions, forcing it to clone the entire
repository just to inspect what files changed.

The pr-reviewer agent already has both permissions. This change aligns the
implementation-worker permissions with its actual usage patterns and reduces
unnecessary full-repo clones in PR fix mode.

Also adds curl, printenv, and echo bash permissions that were inadvertently
omitted from the permission block. These are needed for the agent to make
Forgejo API calls, check environment variables, and perform basic shell
operations.

ISSUES CLOSED: #8175
2026-04-25 04:00:45 +00:00
HAL9000 49ecf7ff8f fix(agents): add PR diff and file list permissions to implementation-worker
The implementation-worker has a PR fix mode that needs to read PR changes
to understand what is failing. However, it lacked forgejo_list_pull_request_files
and forgejo_get_pull_request_diff permissions, forcing it to clone the entire
repository just to inspect what files changed.

The pr-reviewer agent already has both permissions. This change aligns the
implementation-worker permissions with its actual usage patterns and reduces
unnecessary full-repo clones in PR fix mode.

Also adds curl, printenv, and echo bash permissions that were inadvertently
omitted from the permission block. These are needed for the agent to make
Forgejo API calls, check environment variables, and perform basic shell
operations.

ISSUES CLOSED: #8175
2026-04-25 04:00:45 +00:00
HAL9000 1732436edb fix(agents): add PR diff and file list permissions to implementation-worker
The implementation-worker has a PR fix mode that needs to read PR changes to understand what is failing. However, it lacked forgejo_list_pull_request_files and forgejo_get_pull_request_diff permissions, forcing it to clone the entire repository just to inspect what files changed.

The pr-reviewer agent already has both permissions. This change aligns the implementation-worker permissions with its actual usage patterns and reduces unnecessary full-repo clones in PR fix mode.

Also adds curl, printenv, and echo bash permissions that were inadvertently omitted from the permission block restructuring. These are needed for the agent to make Forgejo API calls, check environment variables, and perform basic shell operations.

ISSUES CLOSED: #8175
2026-04-25 04:00:45 +00:00