Skip to content

feat(BOR-476): profile import/export with conflict handling#15

Merged
STRRL merged 5 commits intoSTRRL:mainfrom
bdclaw2026:feat/bor-476-profile-import-export
Mar 8, 2026
Merged

feat(BOR-476): profile import/export with conflict handling#15
STRRL merged 5 commits intoSTRRL:mainfrom
bdclaw2026:feat/bor-476-profile-import-export

Conversation

@bdclaw2026
Copy link
Copy Markdown
Collaborator

@bdclaw2026 bdclaw2026 commented Feb 21, 2026

Summary

  • add versioned profile export payload (version: 1)
  • add profile export with 2 modes: with secrets / without secrets
  • add profile import with 3 conflict strategies: rename / overwrite / skip
  • add import/export actions in sidebar Connections menu
  • show import result summary in UI
  • keep existing profile flows unchanged

Notes

  • lint passes for web app
  • full build is blocked in this environment because wasm-pack is not installed (existing project requirement for @s3-studio/opendal-wasm)

Ticket

  • BOR-476

@bdclaw2026
Copy link
Copy Markdown
Collaborator Author

@codex @claude could you review this PR when you have a moment? Thanks!

@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 21, 2026

@claude @codex review please

@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 21, 2026

https://github.com/codex review please

@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 21, 2026

@codex review please

@STRRL STRRL closed this Feb 22, 2026
@STRRL STRRL reopened this Feb 22, 2026
@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 22, 2026

@codex review please

@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 22, 2026

https://github.com/codex review please

@STRRL
Copy link
Copy Markdown
Owner

STRRL commented Feb 22, 2026

@codex review please

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: a063280ad1

ℹ️ 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 apps/s3-studio/src/App.tsx
Comment thread apps/s3-studio/src/stores/profile-store.ts Outdated
@bdclaw2026
Copy link
Copy Markdown
Collaborator Author

@claude please review this PR

@bdclaw2026
Copy link
Copy Markdown
Collaborator Author

@codex please review 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: a063280ad1

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/s3-studio/src/stores/profile-store.ts Outdated
Comment thread apps/s3-studio/src/stores/profile-store.ts Outdated
- App.tsx: change hasProfiles effect to run only on mount so the
  sidebar import flow is reachable when no profiles exist
- profile-store: preserve existing secretAccessKey/sessionToken when
  overwriting with a sanitized (includeSecrets:false) import
- profile-store: keep activeProfileId as null on import so
  intentionally-disconnected users are not auto-reconnected

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bdclaw2026
Copy link
Copy Markdown
Collaborator Author

@claude please review this PR

@bdclaw2026
Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

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: eed1e90152

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/s3-studio/src/stores/profile-store.ts Outdated
@STRRL STRRL merged commit 58a775e into STRRL:main Mar 8, 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.

2 participants