feat(forge-mcp): expose CI job logs and issue dependency graph #1159

New issue

Closed

opened 2026-05-14 21:27:47 +00:00 by claude-desktop · 0 comments

claude-desktop commented 2026-05-14 21:27:47 +00:00

Collaborator

Copy link

As a dev / code-lead / reviewer agent, I want to read CI job logs and query the issue dependency graph via forge-mcp, so that I can self-diagnose CI failures without operator intervention and check whether my assigned issue is blocked before starting work.

Acceptance criteria

get_job_logs

• Tool get_job_logs(owner?, repo?, forge?, run_id, job_idx) added to SPECS in apps/forge-mcp/src/tools.ts
• Delegates to port.getJobLogs(repo, runId, jobIdx) (already implemented on ForgePort)
• run_id and job_idx are required integer args; owner/repo/forge use the standard repoFragment
• Returns the raw log string; ok() wraps so a missing run emits an MCP error, not null
• get_job_logs added to FORGE_TOOLS_ALLOWLIST in apps/server/src/domain/agent/mcp-config.ts

get_blockers / get_blocked

• Tool get_blockers(owner?, repo?, forge?, index) — delegates to port.getBlockers(repo, index) — returns ForgeIssueRef[]
• Tool get_blocked(owner?, repo?, forge?, index) — delegates to port.getBlocked(repo, index) — returns ForgeIssueRef[]
• Both tools added to FORGE_TOOLS_ALLOWLIST

add_blocker

• Tool add_blocker(owner?, repo?, forge?, index, blocker) — delegates to port.addBlocker(repo, index, blocker) — blocker is an integer issue number
• Added to FORGE_TOOLS_ALLOWLIST

Tests

• TOOL_NAMES lockstep assertion in apps/forge-mcp/src/tools.test.ts includes all 4 new names
• server.test.ts surface count updated (35 → 39)
• mcp-config.test.ts lockstep set updated to require the 4 new names
• just qa passes

Out of scope

• Adapter parity for GitHub/GitLab getJobLogs — adapters that lack the API surface can throw; cross-forge parity is a separate story.
• removeBlocker / dependency bulk-edit — not needed now.
• repoHasWorkflows tool — utility guard, no agent use case yet.

References

• apps/forge-mcp/src/tools.ts — extend SPECS array
• apps/server/src/domain/agent/mcp-config.ts — FORGE_TOOLS_ALLOWLIST
• apps/server/src/infrastructure/forge/forgejo-port.ts lines 374–386 — getBlockers, getBlocked, addBlocker, getJobLogs

As a dev / code-lead / reviewer agent, I want to read CI job logs and query the issue dependency graph via `forge-mcp`, so that I can self-diagnose CI failures without operator intervention and check whether my assigned issue is blocked before starting work. ## Acceptance criteria ### `get_job_logs` - [ ] Tool `get_job_logs(owner?, repo?, forge?, run_id, job_idx)` added to `SPECS` in `apps/forge-mcp/src/tools.ts` - [ ] Delegates to `port.getJobLogs(repo, runId, jobIdx)` (already implemented on `ForgePort`) - [ ] `run_id` and `job_idx` are required integer args; `owner`/`repo`/`forge` use the standard `repoFragment` - [ ] Returns the raw log string; `ok()` wraps so a missing run emits an MCP error, not `null` - [ ] `get_job_logs` added to `FORGE_TOOLS_ALLOWLIST` in `apps/server/src/domain/agent/mcp-config.ts` ### `get_blockers` / `get_blocked` - [ ] Tool `get_blockers(owner?, repo?, forge?, index)` — delegates to `port.getBlockers(repo, index)` — returns `ForgeIssueRef[]` - [ ] Tool `get_blocked(owner?, repo?, forge?, index)` — delegates to `port.getBlocked(repo, index)` — returns `ForgeIssueRef[]` - [ ] Both tools added to `FORGE_TOOLS_ALLOWLIST` ### `add_blocker` - [ ] Tool `add_blocker(owner?, repo?, forge?, index, blocker)` — delegates to `port.addBlocker(repo, index, blocker)` — `blocker` is an integer issue number - [ ] Added to `FORGE_TOOLS_ALLOWLIST` ### Tests - [ ] `TOOL_NAMES` lockstep assertion in `apps/forge-mcp/src/tools.test.ts` includes all 4 new names - [ ] `server.test.ts` surface count updated (35 → 39) - [ ] `mcp-config.test.ts` lockstep set updated to require the 4 new names - [ ] `just qa` passes ## Out of scope - Adapter parity for GitHub/GitLab `getJobLogs` — adapters that lack the API surface can throw; cross-forge parity is a separate story. - `removeBlocker` / dependency bulk-edit — not needed now. - `repoHasWorkflows` tool — utility guard, no agent use case yet. ## References - `apps/forge-mcp/src/tools.ts` — extend `SPECS` array - `apps/server/src/domain/agent/mcp-config.ts` — `FORGE_TOOLS_ALLOWLIST` - `apps/server/src/infrastructure/forge/forgejo-port.ts` lines 374–386 — `getBlockers`, `getBlocked`, `addBlocker`, `getJobLogs`

claude-desktop added the

area:agents

type:user-story

labels 2026-05-14 21:27:50 +00:00

claude-desktop added this to the Forge MCP — multi-forge milestone 2026-05-14 21:27:51 +00:00

code-lead was assigned by architect 2026-05-14 21:28:35 +00:00

code-lead referenced this issue from a commit 2026-05-14 21:38:40 +00:00

feat(forge-mcp): expose CI job logs and issue dependency graph

code-lead referenced this issue from a pull request that will close it, 2026-05-14 21:38:54 +00:00

feat(forge-mcp): expose CI job logs and issue dependency graph #1160

reviewer closed this issue 2026-05-14 21:47:16 +00:00

reviewer referenced this issue from a commit 2026-05-14 21:47:17 +00:00

feat(forge-mcp): expose CI job logs and issue dependency graph (#1160)

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

dev/1213

code-lead/1210

code-lead/1202

code-lead/1201

code-lead/1199

code-lead/1198

dev/1166

dev/1168-error-banner

feat/error-banner-1168

dev/1168

dev/1165

dev/1169-board-redispatch

feat/1169-board-card-redispatch

code-lead/1164

dev/1169

dev/1170

dev/1161

code-lead/1159

code-lead/1150

code-lead/1148

dev/1149

code-lead/1151

dev/1152

code-lead/1147

code-lead/1139

code-lead/1138

code-lead/1137

code-lead/1136

code-lead/1131

code-lead/1122

code-lead/1121

code-lead/1120

code-lead/1119

code-lead/1117

chore/sync-pre-push-from-forge-base

fix/flows-yaml-dispatch-identity

feat/board-tap-to-assign

dev/1107

code-lead/1106

code-lead/1108

dev/1104

code-lead/1103

code-lead/1080

dev/1087

feat/flows-yaml-ci-events

chore/board-drop-stalled-and-density-controls

fix/flows-yaml-routes-always-register

flows-yaml/api-defaults

dev/1023

fix/event-log-history-bleed

fix/janitor-fix-ci-logs-and-cap

dev/1022

fix/board-card-provider

code-lead/1036

dev/1025

code-lead/1020

dev/1017

code-lead/1026

feat/web-shortcut-registry-1018

dev/1015

code-lead/1009

code-lead/1008

dev/975

dev/969

dev/973

dev/967

code-lead/968

code-lead/953

dev/970

dev/976

code-lead/966

code-lead/956

code-lead/951

dev/962

dev/963

dev/977

dev/955

dev/983

dev/961

dev/974

code-lead/950

code-lead/939

dev/941

dev/940

dev/937

dev/938

dev/936

dev/935

feat/web-i18n-fr-locale

feat/spec-editor-ui-polish

chore/drop-legacy-compat

fix/skills-drop-preview-pane

fix/882-skills-safety-rail

dev/911

dev/909

dev/923

dev/917

dev/915

feat/879-sr11-m2-drop-legacy-skill

code-lead/873

dev/881

code-lead/869

dev/867

code-lead/845

code-lead/843

code-lead/844

dev/837

dev/861

dev/849

code-lead/837

code-lead/842

fix/dedup-rebase-inflight

dev/838

code-lead/847

dev/833

code-lead/848

pr/838

code-lead/841

feat/settings-save-bar/836

code-lead/840

dev/846

code-lead/839

dev/832

fix/board-sse-stale-cache

dev/834

dev/835

feat/settings-breadcrumbs

feat/forge-oauth-credentials

refactor/service-config-consolidation

feat/agent-tokens-to-secrets

feat/gitlab-oauth-to-db

feat/authelia-rip-and-voice-fixes

fix/rebase-storm-and-dead-letter

code-lead/797

code-lead/796

dev/811

code-lead/798

dev/810

code-lead/795

dev/808

code-lead/794

dev/805

dev/802

dev/803

feat/avatar-menu-settings-entry

feat/per-agent-token-tracking

dev/793

dev/747

dev/752

code-lead/790

code-lead/759

dev/756

dev/760

dev/741

dev/767

dev/740

dev/709

dev/644

dev/637

boss/614

dev/600

dev/611

dev/585

fix/login-bonus-fixes

boss/544

dev/542

refactor/api-prefix-and-session-gate

dev/489

boss/531

boss/518

dev/499

boss/516

dev/530

dev/517

dev/519

dev/515

dev/522

dev/503

dev/471

boss/329

dev/417

dev/418

dev/402

boss/327

dev/334

dev/332

boss/326

boss/325

dev/331

boss/324

boss/323

boss/322

dev/294

test/s11-task-analytics

dev/262

boss/270

dev/268

foreman/ui-consolidation-spec

dev/234

boss/196

boss/176

boss/164

fix/124-session-persist-bind

boss/52

dev/87

boss/73

dev/77

dev/81

dev/82

boss/79

dev/42

dev/35

boss/7

No results found.

Labels

Clear labels   

area:agents

Agent types, pool scheduling, per-instance config

  

area:dashboard

Dashboard UI and observability surfaces

  

area:database

DB layer — schema, migrations, ORM, raw SQL

  

area:design

UI/UX mockup work — routes to designer agent

  

area:design-review

Design review dispatch — routes to design-reviewer agent

  

area:flows

Flow runner — YAML loader, executor, op registry, expression eval

  

area:infra

Deployment, isolation, containers, systemd units

  

area:meta

Tracking, scaffolding, project setup

  

area:security

Security — routes to reviewer-security (opus)

  

area:sessions

Session-id store, Claude SDK resume logic

  

area:webhook

Forgejo webhook routing and handlers

  

area:workdir

Clone cache, worktrees, git identity

  

security

Security-sensitive issue

  

type:bug

Bug

  

type:chore

Chore

  

type:meta

Tracking or decisions, not implementation work

  

type:user-story

User story

No labels

area:agents

area:dashboard

area:database

area:design

area:design-review

area:flows

area:infra

area:meta

area:security

area:sessions

area:webhook

area:workdir

security

type:bug

type:chore

type:meta

type:user-story

Milestone

Clear milestone

No items

No milestone

Forge MCP — multi-forge

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

code-lead

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

charles/agent-hooks#1159

No description provided.