Skip to content

fix(acp): convert PERMISSION_OPTIONS list to immutable tuple#768

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/code-quality/immutable-permission-options
Open

fix(acp): convert PERMISSION_OPTIONS list to immutable tuple#768
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/code-quality/immutable-permission-options

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Converts the module-level PERMISSION_OPTIONS list to an immutable tuple to prevent accidental mutation at runtime.

Changes

  • Changed PERMISSION_OPTIONS from list to tuple[PermissionOption, ...] in acp_impl/confirmation.py

Related Issue

Addresses findings from #766

Testing

  • Linting passes (make lint)
  • Tests pass (make test)

This PR was automatically generated by the Code Quality Report workflow.

This PR was created by an AI agent (OpenHands) on behalf of the user.

Converts the module-level PERMISSION_OPTIONS list to a tuple to prevent
accidental mutation at runtime.

Addresses item from code quality report.

Related to #766

Co-authored-by: openhands <openhands@all-hands.dev>
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.

0 participants