Conversation
|
|
Caution Review failedThe pull request is closed. WalkthroughIntroduces a TypeScript-first monorepo structure with Turbo/tsup, adds @tm/core and @tm/cli packages, migrates CLI commands (list/show/auth/context) to Commander classes, implements core config/storage/auth/services, refactors assets handling, updates CI workflow into split jobs, adjusts MCP/extension builds, and adds extensive tests/docs/reports. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User
participant CLI as CLI Command (list/show)
participant Core as TaskMasterCore
participant Svc as TaskService
participant SF as StorageFactory
participant FS as FileStorage
participant API as ApiStorage
User->>CLI: Run command with options
CLI->>Core: createTaskMasterCore(projectPath)
Core->>Svc: initialize()
Svc->>SF: create(config, projectPath)
alt API configured
SF-->>Svc: ApiStorage
Svc->>API: loadTasks(tag)
else File storage
SF-->>Svc: FileStorage
Svc->>FS: loadTasks(tag)
end
Svc-->>Core: TaskListResult / Task
Core-->>CLI: Results
CLI-->>User: Render (text/json/compact)
sequenceDiagram
autonumber
actor User
participant CLI as AuthCommand
participant AM as AuthManager
participant OA as OAuthService
participant SB as SupabaseAuthClient
participant CS as CredentialStore
User->>CLI: auth login
CLI->>AM: getInstance()
CLI->>OA: authenticate({ callbacks })
OA->>SB: signInWithPKCE()
SB-->>OA: auth URL
OA-->>CLI: onAuthUrl / open browser
User-->>OA: Complete browser auth
OA->>SB: exchangeCodeForSession()
SB-->>OA: Session
OA->>CS: saveCredentials()
OA-->>CLI: AuthCredentials
CLI-->>User: Success + status
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120–180 minutes Possibly related PRs
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. ✨ Finishing touches
🧪 Generate unit tests
Comment |
This PR was automatically generated to update documentation based on recent changes. Original commit: feat: add tm show (#1199)\n\n\n Co-authored-by: Claude <claude-assistant@anthropic.com>
What type of PR is this?
Description
Related Issues
How to Test This
# Example commands or stepsExpected result:
Contributor Checklist
npm run changesetnpm testnpm run format-check(ornpm run formatto fix)Changelog Entry
For Maintainers
Summary by CodeRabbit