Skip to content

feat(sdk): add getWorkflowJobLogs, PR review thread tools, and deleteBranch - #115

Merged
HugoRCD merged 1 commit into
mainfrom
feat/ci-logs-review-threads-delete-branch
Aug 28, 2026
Merged

feat(sdk): add getWorkflowJobLogs, PR review thread tools, and deleteBranch#115
HugoRCD merged 1 commit into
mainfrom
feat/ci-logs-review-threads-delete-branch

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

Adds five tools (79 → 84), each shaped to keep model context lean:

  • getWorkflowJobLogs (read) — fetches a workflow job's log and returns only the last maxLines lines (default 200, max 2000), with the per-line ISO timestamps GitHub prefixes stripped out. The response reports totalLines / returnedLines / omittedLines so the model knows what it isn't seeing.
  • listPullRequestReviewThreads (read, GraphQL) — review threads with resolution state and the IDs needed to act: threadId for resolving, commentId for replying. Defaults to unresolved threads only, comment bodies truncated at ~500 chars (detail: 'summary'), cursor-paginated.
  • replyToReviewComment (write) — reply inside a review thread.
  • resolveReviewThread (write, GraphQL) — mark a thread resolved. Idempotent.
  • deleteBranch (write) — permanently delete a branch.

All three writes require approval by default and are registered in GITHUB_WRITE_TOOLS.

Registration & presets

  • Registered across tool-names, write-tools, index.ts, the eve registry, Connect scopes (actions:read, pull_requests:read/write, contents:write — no preset scope changes needed), and chat app metadata.
  • Presets: ci-ops / repo-explorer / security-audit gain getWorkflowJobLogs; code-review gains the three review-thread tools; pr-author gains review threads + deleteBranch; maintainer gains all five. Preset system prompts updated accordingly.
  • Docs: tools catalog, approval-control risk table, presets guide, working-context lean-defaults table, README tables, published skill, and the 84-tool count everywhere.

Test plan

  • New unit tests for the log shaping helper (shapeJobLog): timestamp stripping, tail truncation with omitted-count reporting, CRLF normalization
  • Existing eve registry parity test now covers the five new tools (84 = 84)
  • pnpm build && pnpm lint && pnpm typecheck && pnpm test all pass

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
github-tools-docs Ready Ready Preview, v0 Aug 28, 2026 6:20pm
github-tools-test-agent Ready Ready Preview, v0 Aug 28, 2026 6:20pm

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

@HugoRCD
HugoRCD merged commit d5ea800 into main Aug 28, 2026
15 checks passed
@HugoRCD
HugoRCD deleted the feat/ci-logs-review-threads-delete-branch branch August 28, 2026 18:34
@github-actions github-actions Bot mentioned this pull request Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant