Skip to content

fix: recover pending tool permissions - #29

Merged
theblazehen merged 1 commit into
mainfrom
fix/recover-tool-permissions
Jun 28, 2026
Merged

fix: recover pending tool permissions#29
theblazehen merged 1 commit into
mainfrom
fix/recover-tool-permissions

Conversation

@theblazehen

Copy link
Copy Markdown
Owner

Summary

  • recover pending tool permissions from server state on message load and SSE reconnect
  • support legacy and v2 permission APIs, with v2 reply falling back when old servers return the web UI HTML
  • route approval buttons through permission request ids while still matching recovered prompts to tool call ids
  • add regression tests for v2 mapping, legacy/v2 reconciliation, and stale permission clearing

Verification

  • export JAVA_HOME=/usr/lib/jvm/java-17-openjdk && ./gradlew :app:testDebugUnitTest --tests dev.blazelight.p4oc.data.remote.mapper.EventMapperTest --tests dev.blazelight.p4oc.data.session.SessionRepositoryImplTest
  • export JAVA_HOME=/usr/lib/jvm/java-17-openjdk && ./gradlew :app:compileDebugKotlin
  • installed debug APK on device 192.168.24.119:47293 and verified recovered approval controls for a missed bash permission
  • verified tapping Allow posts legacy fallback /permission/{id}/reply on opencode 1.15.10 and UI renders completed tool output

Fixes #15

@theblazehen
theblazehen merged commit be9478e into main Jun 28, 2026
2 checks passed
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.

tool approval not working/showing

1 participant