Skip to content

[codex] Improve auth status report#16

Merged
dubscode merged 5 commits intomainfrom
issue-9
May 9, 2026
Merged

[codex] Improve auth status report#16
dubscode merged 5 commits intomainfrom
issue-9

Conversation

@dubscode
Copy link
Copy Markdown
Contributor

@dubscode dubscode commented May 9, 2026

Adds a richer linear auth status report with live user, workspace, and configured default team details while preserving JSON envelope output. Exposes profile lookup through AuthManager so the CLI can resolve saved profile settings. The default-team lookup now paginates team results when resolving a configured key, avoiding false nulls in larger workspaces. Validated with pnpm verify.

Copilot AI review requested due to automatic review settings May 9, 2026 04:41
@dubscode dubscode marked this pull request as ready for review May 9, 2026 04:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Enhances the CLI’s linear auth status to produce a richer status report by augmenting the existing auth-status payload with live user/workspace data and resolving a configured default team, while keeping the JSON envelope structure for machine output. This is supported by exposing profile lookup through AuthManager and adding CLI-side report building + tests.

Changes:

  • Added AuthManager.getProfile() (linear-core) and corresponding tests.
  • Introduced buildAuthStatusReport() (cli) to combine base auth status with live viewer/organization details and default-team resolution (including pagination fallback).
  • Updated auth status command to output the richer report (JSON envelope preserved; new human rendering) and added CLI tests.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/linear-core/src/auth/session.ts Adds AuthManager.getProfile() to expose stored profile configuration via the manager.
packages/linear-core/tests/session.test.ts Adds test coverage for AuthManager.getProfile() behavior (default + missing profile).
packages/cli/src/auth/status-report.ts New report builder that enriches auth status with live user/workspace and default team resolution (paginated fallback).
packages/cli/src/index.ts Wires auth status to the new report builder and adds custom human output tables (JSON envelope unchanged).
packages/cli/tests/auth-status.test.ts Adds tests validating report enrichment and fallback behaviors (including pagination).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/cli/src/auth/status-report.ts Outdated
Comment thread packages/cli/src/index.ts Outdated
Comment thread packages/cli/tests/auth-status.test.ts Outdated
Copilot AI review requested due to automatic review settings May 9, 2026 04:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread packages/cli/src/auth/status-report.ts
@dubscode dubscode enabled auto-merge (squash) May 9, 2026 04:56
@dubscode dubscode merged commit 0d74459 into main May 9, 2026
5 checks passed
@dubscode dubscode deleted the issue-9 branch May 9, 2026 04:56
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

🎉 This PR is included in version 1.5.0-alpha.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants