team-board/picogent
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: remove slug params from API calls — Tracker resolves actor from auth

Browse Source
This commit is contained in:
Markov 2026-02-24 13:32:31 +01:00
parent fa9f280bdd
commit 4b3e4d6195
2 changed files with 15 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/tools/tasks.ts
View File

@ -90,7 +90,7 @@ export function createTaskTools(ctx: ToolContext): ToolDefinition<any>[] {
parameters: UpdateTaskParams, parameters: UpdateTaskParams,
async execute(_id: string, params: any) { async execute(_id: string, params: any) {
const { task_id, ...fields } = params; const { task_id, ...fields } = params;
await ctx.trackerClient.updateTask(task_id, fields, ctx.agentSlug); await ctx.trackerClient.updateTask(task_id, fields);
return ok(`Task ${task_id} updated`); return ok(`Task ${task_id} updated`);
}, },
}, },
@ -101,7 +101,7 @@ export function createTaskTools(ctx: ToolContext): ToolDefinition<any>[] {
parameters: TakeTaskParams, parameters: TakeTaskParams,
async execute(_id: string, params: any) { async execute(_id: string, params: any) {
ctx.selfAssignedTasks.add(params.task_id); ctx.selfAssignedTasks.add(params.task_id);
await ctx.trackerClient.takeTask(params.task_id, ctx.agentSlug); await ctx.trackerClient.takeTask(params.task_id);
return ok(`Task ${params.task_id} taken by ${ctx.agentSlug}`); return ok(`Task ${params.task_id} taken by ${ctx.agentSlug}`);
}, },
}, },
@ -111,7 +111,7 @@ export function createTaskTools(ctx: ToolContext): ToolDefinition<any>[] {
description: 'Reject an assigned task with a reason. Task returns to backlog.', description: 'Reject an assigned task with a reason. Task returns to backlog.',
parameters: RejectTaskParams, parameters: RejectTaskParams,
async execute(_id: string, params: any) { async execute(_id: string, params: any) {
await ctx.trackerClient.rejectTask(params.task_id, ctx.agentSlug, params.reason); await ctx.trackerClient.rejectTask(params.task_id, params.reason);
return ok(`Task ${params.task_id} rejected: ${params.reason}`); return ok(`Task ${params.task_id} rejected: ${params.reason}`);
}, },
}, },
@ -121,7 +121,7 @@ export function createTaskTools(ctx: ToolContext): ToolDefinition<any>[] {
description: 'Subscribe to task notifications (become a watcher).', description: 'Subscribe to task notifications (become a watcher).',
parameters: WatchTaskParams, parameters: WatchTaskParams,
async execute(_id: string, params: any) { async execute(_id: string, params: any) {
await ctx.trackerClient.watchTask(params.task_id, ctx.agentSlug); await ctx.trackerClient.watchTask(params.task_id);
return ok(`Now watching task ${params.task_id}`); return ok(`Now watching task ${params.task_id}`);
}, },
}, },

23
src/tracker/client.ts
View File

@ -62,28 +62,27 @@ export class TrackerClient {
return this.request('POST', `/api/v1/tasks?project_slug=${encodeURIComponent(projectSlug)}`, task); return this.request('POST', `/api/v1/tasks?project_slug=${encodeURIComponent(projectSlug)}`, task);
} }
async updateTask(taskId: string, fields: Record<string, unknown>, actor?: string): Promise<void> { async updateTask(taskId: string, fields: Record<string, unknown>): Promise<void> {
this.log.info({ taskId, fields, actor }, 'Updating task'); this.log.info({ taskId, fields }, 'Updating task');
const qs = actor ? `?actor=${encodeURIComponent(actor)}` : ''; await this.request('PATCH', `/api/v1/tasks/${taskId}`, fields);
await this.request('PATCH', `/api/v1/tasks/${taskId}${qs}`, fields);
} }
async deleteTask(taskId: string): Promise<void> { async deleteTask(taskId: string): Promise<void> {
await this.request('DELETE', `/api/v1/tasks/${taskId}`); await this.request('DELETE', `/api/v1/tasks/${taskId}`);
} }
async takeTask(taskId: string, slug: string): Promise<void> { async takeTask(taskId: string): Promise<void> {
this.log.info({ taskId, slug }, 'Taking task'); this.log.info({ taskId }, 'Taking task');
await this.request('POST', `/api/v1/tasks/${taskId}/take?slug=${encodeURIComponent(slug)}`); await this.request('POST', `/api/v1/tasks/${taskId}/take`);
} }
async rejectTask(taskId: string, slug: string, reason?: string): Promise<void> { async rejectTask(taskId: string, reason?: string): Promise<void> {
this.log.info({ taskId, slug, reason }, 'Rejecting task'); this.log.info({ taskId, reason }, 'Rejecting task');
await this.request('POST', `/api/v1/tasks/${taskId}/reject`, { slug, reason }); await this.request('POST', `/api/v1/tasks/${taskId}/reject`, { reason });
} }
async watchTask(taskId: string, slug: string): Promise<void> { async watchTask(taskId: string): Promise<void> {
await this.request('POST', `/api/v1/tasks/${taskId}/watch?slug=${encodeURIComponent(slug)}`); await this.request('POST', `/api/v1/tasks/${taskId}/watch`);
} }
// --- Steps (checklist inside task) --- // --- Steps (checklist inside task) ---