Skip to content

Simplify project context/auth flow for read-only commands + persist project_id#58

Merged
Parker Henderson (parkerhendo) merged 4 commits intomainfrom
make-faster
Apr 10, 2026
Merged

Simplify project context/auth flow for read-only commands + persist project_id#58
Parker Henderson (parkerhendo) merged 4 commits intomainfrom
make-faster

Conversation

@parkerhendo
Copy link
Copy Markdown
Contributor

@parkerhendo Parker Henderson (parkerhendo) commented Mar 2, 2026

Changes

  1. bt switch now persists project_id in config alongside org and project name
  2. Read-only commands (list/view operations) use fast authentication path when cached project_id exists, avoiding full login validation
  3. Added login_read_only() and fast_login() functions to support optimized authentication for read operations
  4. Experiments, prompts, and functions modules now route read-only commands through optimized auth path
  5. Created shared ProjectContext type and resolve_project_context() function for consistent project resolution
  6. Spinner display delay increased from 300ms to 500ms
  7. Added comprehensive test coverage for new authentication modes and project context resolution

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 2, 2026

Latest downloadable build artifacts for this PR commit f18dad5eb538:

Available artifact names
  • ``artifacts-build-global
  • ``artifacts-build-local-x86_64-apple-darwin
  • ``artifacts-build-local-x86_64-pc-windows-msvc
  • ``artifacts-build-local-x86_64-unknown-linux-musl
  • ``artifacts-build-local-x86_64-unknown-linux-gnu
  • ``artifacts-build-local-aarch64-unknown-linux-musl
  • ``artifacts-build-local-aarch64-apple-darwin
  • ``artifacts-build-local-aarch64-unknown-linux-gnu
  • ``artifacts-plan-dist-manifest
  • ``cargo-dist-cache

@ibolmo
Copy link
Copy Markdown
Contributor

happy to try to review (rust is not yet my strong suit), but at a high level this pr could use some tests. i'll come back around today, but let me know if you'd like to proceed without tests Parker Henderson (@parkerhendo)

@parkerhendo
Copy link
Copy Markdown
Contributor Author

happy to try to review (rust is not yet my strong suit), but at a high level this pr could use some tests. i'll come back around today, but let me know if you'd like to proceed without tests Parker Henderson (Parker Henderson (@parkerhendo))

Olmo Maldonado (@ibolmo) I'll go back through and add some tests for it.

Copy link
Copy Markdown
Contributor Author

Olmo Maldonado (@ibolmo) added some tests for the new paths.

@parkerhendo Parker Henderson (parkerhendo) merged commit b7fca91 into main Apr 10, 2026
32 of 34 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