feat: add skill install command for coding agent integrations#16
Merged
Conversation
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
|
In a follow-up PR, we could add support for more agents (get inspired by https://github.com/vercel-labs/add-skill) – I'd at least add Codex as it's used frequently at Doist 👍 |
henningmu
approved these changes
Jan 23, 2026
scottlovegrove
approved these changes
Jan 23, 2026
Collaborator
Author
That is precisely the idea. That's why I architected it with multi-agent in mind. It's now up for others to add support for their agentic app of their choice. |
2 tasks
9 tasks
scottlovegrove
added a commit
that referenced
this pull request
May 12, 2026
) ## Summary Bumps `@doist/cli-core` to **0.10.0** and migrates `td auth logout` + `td auth status` onto the new `attachLogoutCommand` / `attachStatusCommand` registrars that ship alongside the existing `attachLoginCommand` ([cli-core#16](Doist/cli-core#16)). - `TodoistTokenStore` gains `getLastClearResult()` so the logout `onCleared` callback can surface keyring-fallback warnings — cli-core's `TokenStore.clear: void` contract can't carry the `TokenStorageResult` directly. - `auth/index.ts` now creates a single shared `TodoistTokenStore` instance used by login, logout, and status (login's wrapper takes the store as a parameter instead of constructing its own). - `attachTodoistStatusCommand` routes both the active-snapshot path (`fetchLive`) and the unauthenticated path (`onNotAuthenticated`) through one `gatherStatusData` helper so env-token mode and `--user <ref>` (both return `null` from `TokenStore.active()` by the adapter's documented contract) render byte-for-byte identically to the persisted-default-user path. - `auth logout` and `auth status` now accept `--ndjson` for free (framework-registered) on top of `--json`. `auth status` previously only had `--json`. - `td auth token view` stays hand-rolled — its `--user <ref>` selector depends on multi-user state the `TokenStore` adapter intentionally drops from `active()`. cli-core PR #16 explicitly carves this out as a scope note pending a multi-user store contract. ## Test plan - [x] `npm run check` (oxlint + oxfmt) clean - [x] `npm run type-check` clean - [x] `npm test` — 1574/1574 across 60 files - [x] `npm run build` clean - [x] Smoke: `HOME=$(mktemp -d) td auth status` prints the legacy "No API token found." CliError envelope - [x] Smoke: `td auth {login,logout,status} --help` show framework-registered `--json` / `--ndjson` flags - [ ] Manual: full OAuth login → `td auth status` → `td auth status --json` → `td auth logout` round-trip against a real account - [ ] Manual: `TODOIST_API_TOKEN=… td auth status` shows `✓ Authenticated (TODOIST_API_TOKEN)` env-mode branch - [ ] Manual: multi-user setup shows the "Other stored accounts" enumeration 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
td skill install <agent>command to install skill files for coding agentsclaude-codeagent (installs SKILL.md with Todoist CLI documentation)Commands
Test plan
npm run buildcompiles successfullynpm testpasses (580 tests)td skill listshows available agentstd skill install claude-codecreates skill filetd skill install claude-codeerrors without--forcewhen file existstd skill uninstall claude-coderemoves skill file🤖 Generated with Claude Code