agent: route task tools through tk- displayIds#882
Closed
harshithmullapudi wants to merge 2 commits into
Closed
Conversation
Agent tools now accept tk- displayIds (e.g. tk-abcde, tk-abcde.1) in
addition to UUIDs for every task lookup. Internally the new
resolveTaskId(input, workspaceId) helper detects the format and returns
the canonical UUID, scoped to the workspace via the existing
(workspaceId, displayId) unique index.
Threaded through:
- task-tools: get_task, update_task, unblock_task, delete_task,
confirm_task, get_task_tree (taskId), create_task (parentTaskId),
update_task (newParentId)
- session-tools: get_task_coding_session, list_task_coding_sessions,
list_task_browser_sessions
Tool outputs now render displayId so the agent has a stable, human-
readable handle to round-trip. UUID remains the internal PK; no schema
changes. reschedule_self is untouched (uses server-injected UUID).
Also includes unrelated WIP bundled at the user's request:
decision-agent prompt cleanup, granola/ynab SDK bumps, GitHub
integration tool-UI flags, and granola schedule formatting.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…gents
Removed the misleading example names ("codex", "cursor-agent") from the
CODING AGENT SELECTION block — they don't match the registered names
(e.g. the real one is codex-cli), so the model was passing user wording
verbatim and skipping the lookup.
Now the prompt requires calling coding_list_agents when the intent names
a specific agent, then passing the exact registered name to coding_ask.
The default path (no agent specified) still skips the lookup.
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
1 participant
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
tk-…displayIds (e.g.tk-abcde,tk-abcde.1) for every task lookup, in addition to UUIDs. NewresolveTaskId(input, workspaceId)helper auto-detects the format and returns the canonical UUID via the existing(workspaceId, displayId)unique index — no schema or PK changes.task-tools(get_task,update_task,unblock_task,delete_task,confirm_task,get_task_tree, plusparentTaskId/newParentIdoncreate_task/update_task) andsession-tools(get_task_coding_session,list_task_coding_sessions,list_task_browser_sessions).displayId ?? idso the agent has a stable, human-readable handle to round-trip.reschedule_selfuntouched (uses server-injected UUID).Test plan
tsc --noEmit— zero new errors introduced in the changed files (verified by stashing and re-running; same 12 pre-existing errors)vitest run app/services/agent/__tests__/session-tools.test.ts— 7/7 passingtk-xxxxxtoget_taskreturns the task; confirmupdate_taskreparent reports the new displayIdget_task_coding_sessionwith atk-…arg resolves correctly🤖 Generated with Claude Code