Skip to content

fix(auth): preserve remote step 2 replay flags#427

Merged
steipete merged 2 commits intoopenclaw:mainfrom
doodaaatimmy-creator:codex/remote-auth-step2-flags
Mar 7, 2026
Merged

fix(auth): preserve remote step 2 replay flags#427
steipete merged 2 commits intoopenclaw:mainfrom
doodaaatimmy-creator:codex/remote-auth-step2-flags

Conversation

@doodaaatimmy-creator
Copy link
Copy Markdown
Contributor

Summary

  • preserve the scope-shaping flags in remote auth step-2 guidance so replaying the manual flow does not drop required options
  • cover the new replay instruction in the existing auth command test

Fixes #405.

Testing

  • go test ./internal/cmd -run "TestAuthAddCmd_RemoteStep1_PrintsAuthURL|TestAuthAddCmd_RemoteStep2_PassesAuthURL|TestAuthAddCmd_RemoteStep2_RejectsAuthCode"
  • go test ./internal/googleauth -run "TestManualAuthURL_ReusesState"

@steipete steipete force-pushed the codex/remote-auth-step2-flags branch from f160fa0 to a59086d Compare March 7, 2026 22:46
@steipete steipete merged commit e50dba7 into openclaw:main Mar 7, 2026
2 of 9 checks passed
@steipete
Copy link
Copy Markdown
Collaborator

steipete commented Mar 7, 2026

Landed via rebase onto main.

  • Gate: go test ./internal/cmd -run 'TestAuthAddCmd_RemoteStep1_PrintsAuthURL|TestAuthAddCmd_RemoteStep2_PassesAuthURL|TestAuthAddCmd_RemoteStep2_RejectsAuthCode' && make test
  • Land commit: a59086d
  • Merge commit: e50dba7

Thanks @doodaaatimmy-creator!

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.

Remote step 1 output should include --services/--readonly flags in step 2 instructions

2 participants