Skip to content

feat(cli): add unified potpie atlassian login flow (2/2)#913

Open
Nih1tGupta wants to merge 19 commits into
feat/bitbucket-cli-foundationfrom
feat/atlassian-suite-stacked
Open

feat(cli): add unified potpie atlassian login flow (2/2)#913
Nih1tGupta wants to merge 19 commits into
feat/bitbucket-cli-foundationfrom
feat/atlassian-suite-stacked

Conversation

@Nih1tGupta

Copy link
Copy Markdown
Contributor

Closes #911

Part 2 of 2. Builds on #912 (Bitbucket foundation).

Summary

  • Adds potpie atlassian login|logout|status for Jira + Confluence + Bitbucket in one guided flow.
  • Refactors atlassian_auth.py with shared connect helpers used by the suite.

What’s new

  • atlassian login — Jira → Confluence → Bitbucket (skips already-connected unless --force)
  • atlassian logout — clears all three
  • atlassian status — combined auth snapshot
  • --skip-bitbucket and non-interactive credential flags supported

potpie jira, potpie confluence, and potpie bitbucket remain unchanged.

Dependencies

Test plan

  • pytest tests/unit/test_cli_atlassian_suite.py tests/unit/test_cli_atlassian.py tests/unit/test_cli_atlassian_shared.py tests/unit/test_bitbucket_*.py -q (19 suite tests + existing Atlassian/Bitbucket tests)
  • Manual: potpie atlassian login end-to-end
  • Manual: --skip-bitbucket, status, logout
  • Manual: standalone potpie jira login / potpie bitbucket login still work
Screenshot 2026-06-22 at 5 31 08 PM

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5e026afe-ecb6-47d2-8394-5c7b9018f121

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Nihit Gupta and others added 18 commits June 23, 2026 10:49
…ce, record routing, nudge (#914)

Layer 2 of the linear CE stack (on top of cg-05 backends, now on main). Adds the
application-service tier over the backends: the graph workbench (plan/inbox/
history/quality persistence + dispatcher), the V2 read surface (7 readers +
read-orchestrator + agent-context/envelope builder), record routing
(record→semantic bridge, durable-context use case, context-graph jobs), the
nudge service, pot management, setup/ingestion orchestrators, and the
context-graph HTTP router + outbound API client.

53 owned files byte-identical to feat/graph-updates, including the cg-05
forward-dep test carry-in restored to full gu (test_graph_backend_conformance,
benchmarks/retrieval_eval + test_retrieval_eval, test_context_graph_writer).

Standalone-green reconciliations (this layer introduces strict record validation
+ the owners reader, whose matching conformance updates are owned by cg-11):
- co-own tests/conformance/test_local_profile_completion.py from gu (byte-identical)
- bridge tests/conformance/test_host_shell_end_to_end.py (cg-11-owned): add the
  required record details + retarget the unsupported-include test to an unknown
  include (every advertised family now has a reader), keeping cg-06's pre-ledger-
  read-only API. Transient — superseded when cg-11 restores gu's version.
* feat: CE detached daemon refactor on top of #889 (daemon_rpc/daemon_main/runtime; supersede old daemon tests)

* test: align FalkorDB default assertion with Python version

* fix: address daemon review findings

* fix: avoid double-closing daemon pidfile fd

---------

Co-authored-by: nndn <mynameisgurunandan@gmail.com>
#918)

* feat: CE graph explorer UI and repo resolution

* fix: align graph skill catalog title

* fix: address graph ui review feedback

* feat: mount UI

---------

Co-authored-by: nndn <mynameisgurunandan@gmail.com>
…reorg (#919)

Co-authored-by: nndn <mynameisgurunandan@gmail.com>
* Remove observability package

* Fix observability logging review comments

* Fix legacy logging interpolation regression
…#876)

* Bake Linear and GitHub OAuth client IDs into CLI wheel at build time.

Ship public OAuth client IDs via a Hatch build hook so end users can run
potpie setup and integration login without setting LINEAR_CLIENT_ID or
POTPIE_GITHUB_CLIENT_ID in their own environment.

Co-authored-by: Cursor <cursoragent@cursor.com>

* Address PR review: centralize baked OAuth config and rename build hook.

Use baked_oauth_client_ids module instead of inline imports, rename the
Hatch hook to oauth_client_id_injection_hook, remove committed _build_config
stub (generated only at wheel build), and improve missing-client-ID errors.

Co-authored-by: Cursor <cursoragent@cursor.com>

* Simplify OAuth client ID errors for OSS end users.

Show .env setup guidance only in local development; otherwise prompt users
to report the issue on GitHub instead of exposing build-time wiring details.

Co-authored-by: Cursor <cursoragent@cursor.com>

* Rename OAuth client ID module, simplify resolution/errors, and fix gitignore path.

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
* Disable keyring credential storage

* Rename credential store

* Write credential files atomically
* Preserve existing agent instructions

* Address CodeRabbit agent instruction comments

* Handle embedded unmarked Potpie instructions

---------

Co-authored-by: nndn <mynameisgurunandan@gmail.com>
* version fix

* b1 for ce
Co-authored-by: nndn <mynameisgurunandan@gmail.com>
@Nih1tGupta Nih1tGupta changed the title feat(cli): add unified potpie atlassian login flow feat(cli): add unified potpie atlassian login flow (2/2) Jun 24, 2026
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.

4 participants