Skip to content

Add guided Relayfile cloud setup#70

Open
khaliqgant wants to merge 1 commit intomainfrom
codex/relayfile-guided-cloud-setup
Open

Add guided Relayfile cloud setup#70
khaliqgant wants to merge 1 commit intomainfrom
codex/relayfile-guided-cloud-setup

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

Summary

  • make bare relayfile and relayfile setup run the guided Cloud onboarding flow
  • add browser Cloud login, workspace create/join, Nango connect session polling, credential/catalog persistence, and handoff to the existing mount loop
  • document the new first-run flow and add focused CLI coverage

Tests

  • go test ./...

Note: unrelated local trajectory index/untracked trajectory files were left out of this PR.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 38eef7daec

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread cmd/relayfile-cli/main.go
}
}

cloudClient, err := newAPIClient(cloudAPI, tokenSet.AccessToken)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Use login callback API URL for subsequent Cloud requests

runCloudLogin parses and returns api_url from the callback (via readCloudCredentialsFromQuery), but setup still constructs cloudClient with the original cloudAPI value. If Cloud login redirects through a front door and returns a different API origin (regional/tenant-specific), workspace creation/join requests are sent to the wrong host and setup fails after successful auth. Use tokenSet.APIURL (or overwrite cloudAPI after login) before creating the API client.

Useful? React with 👍 / 👎.

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.

1 participant