f6cc2f1c40Implements the final two slices of the OpenSpec change
mcp-resource-templates. The change now ships all 7 entity
resources (owner, repo, commit, status, issue, comment, pr) on
the forgejo:// URI scheme.
Slice 6 — mcp-resource-pr
- operation/pull/resources.go registers
forgejo://repo/{owner}/{repo}/pr/{index}
- Handler returns JSON metadata (head/base refs, mergeability,
state) + text/markdown sidecar + bounded recent_comments and
recent_reviews (cap 30, sentinels name list_issue_comments
and list_pull_reviews respectively)
- Non-numeric index → -32602; missing PR → -32003
- 6 test cases: open/merged PR, > cap comments, > cap reviews,
non-numeric index, 404
- Wired from RegisterCoreResources
Slice 7 — docs/wrap
- README "Resources" section enriched with URI scheme rationale,
the coexist-with-tools rule, and the embedded-list bounding
policy
- AGENTS.md "## Resources" subsection added for future AI
assistants modifying the resource surface
- CHANGELOG.md unreleased section: additive resource surface
- Follow-up beads filed:
- forgejo-mcp-7ra: revisit subscribe=true once a real
subscription use case appears
- forgejo-mcp-7de: revisit EmbeddedListCap=30 once telemetry
available
Tasks 6.1-6.5, 7.1-7.6 ticked. Final tick count: 46/47.
Only 1.11 (manual client verification across Claude
Code/Desktop/Codex/Cursor) remains, out of scope for this team.
Driven by team:dev-loop (impl, runner; impl replaced by impl2
mid-slice-6 after the original impl ignored two checkpoint
probes). Closes the implementation portion of forgejo-mcp-13x.
123 lines
4.6 KiB
Markdown
123 lines
4.6 KiB
Markdown
# AGENTS.md
|
|
|
|
This file provides guidance to AI coding assistants (Claude Code, Cursor, etc.) when working with this repository.
|
|
|
|
For detailed developer documentation, see [DEVELOPER.md](DEVELOPER.md).
|
|
|
|
## Quick Reference
|
|
|
|
```bash
|
|
make build # Build the binary (outputs ./forgejo-mcp)
|
|
make vendor # Tidy and verify Go module dependencies
|
|
```
|
|
|
|
## Architecture Summary
|
|
|
|
```
|
|
main.go → cmd/cmd.go (CLI parsing) → operation/operation.go (tool registration) → operation/{domain}/*.go (tool handlers)
|
|
```
|
|
|
|
Key directories:
|
|
- `operation/` - MCP tool definitions and handlers by domain
|
|
- `pkg/forgejo/` - Singleton Forgejo SDK client wrapper
|
|
- `pkg/to/` - Response formatting helpers
|
|
- `pkg/params/` - Shared parameter descriptions
|
|
|
|
## Adding a New Tool
|
|
|
|
1. Create or modify a file in `operation/{domain}/`
|
|
2. Define tool with `mcp.NewTool()` and implement handler function
|
|
3. Register in the domain's `RegisterTool(s *server.MCPServer)` function
|
|
4. If new domain, import and call in `operation/operation.go`
|
|
5. **Bound the output.** If response size depends on data (not tool
|
|
semantics), the tool MUST satisfy [docs/design/output-bounding.md](docs/design/output-bounding.md):
|
|
client-controlled bound + resumability + documented parameters. Use the
|
|
checklist there in the PR description.
|
|
|
|
See [DEVELOPER.md](DEVELOPER.md) for complete code examples and patterns.
|
|
|
|
## Resources
|
|
|
|
Resource templates expose Forgejo entities as `forgejo://` URIs — instance-portable, additive, coexisting with all existing tools (no tool removed). Clients that support `resources/templates/list` and `resources/read` resolve these URIs directly; others fall back to tools transparently.
|
|
|
|
When adding a new resource template, place it under `operation/<domain>/resources*.go`. Use the `operation/resource` package for URI parsing (`ParseXxx`), embedded-list bounding (`Bounded`), and error mapping (`MapForgejoError`). Embedded lists MUST use `operation/resource.Bounded` so the truncation sentinel stays consistent across resources.
|
|
|
|
See `openspec/specs/mcp-resources-core/spec.md` for the full normative spec (added by this slice when the change archives).
|
|
|
|
## Blocked Features
|
|
|
|
Some features are blocked on upstream API/SDK support. See `docs/plans/` for:
|
|
|
|
- `wiki-support.md` - Wiki API (blocked on forgejo-sdk)
|
|
- `projects-support.md` - Projects/Kanban API (blocked on Gitea 1.26.0)
|
|
|
|
## Repository Labels
|
|
|
|
Labels for goern/forgejo-mcp on Codeberg:
|
|
|
|
| ID | Name | Color | Description |
|
|
|----|------|-------|-------------|
|
|
| 335058 | Kind/Feature | 0288d1 | New functionality |
|
|
| 335061 | Kind/Enhancement | 84b6eb | Improve existing functionality |
|
|
| 335091 | Status/Blocked | 880e4f | Something is blocking this issue or pull request |
|
|
| 335103 | Priority/Medium | e64a19 | The priority is medium |
|
|
|
|
### Usage with Codeberg MCP
|
|
|
|
When adding labels via the `mcp__codeberg__add_issue_labels` tool, use the numeric ID:
|
|
|
|
```
|
|
mcp__codeberg__add_issue_labels(
|
|
owner: "goern",
|
|
repo: "forgejo-mcp",
|
|
index: <issue_number>,
|
|
labels: "<label_id>" # e.g., "335091" for Status/Blocked
|
|
)
|
|
```
|
|
|
|
<!-- BEGIN BEADS INTEGRATION v:1 profile:minimal hash:ca08a54f -->
|
|
## Beads Issue Tracker
|
|
|
|
This project uses **bd (beads)** for issue tracking. Run `bd prime` to see full workflow context and commands.
|
|
|
|
### Quick Reference
|
|
|
|
```bash
|
|
bd ready # Find available work
|
|
bd show <id> # View issue details
|
|
bd update <id> --claim # Claim work
|
|
bd close <id> # Complete work
|
|
```
|
|
|
|
### Rules
|
|
|
|
- Use `bd` for ALL task tracking — do NOT use TodoWrite, TaskCreate, or markdown TODO lists
|
|
- Run `bd prime` for detailed command reference and session close protocol
|
|
- Use `bd remember` for persistent knowledge — do NOT use MEMORY.md files
|
|
|
|
## Session Completion
|
|
|
|
**When ending a work session**, you MUST complete ALL steps below. Work is NOT complete until `git push` succeeds.
|
|
|
|
**MANDATORY WORKFLOW:**
|
|
|
|
1. **File issues for remaining work** - Create issues for anything that needs follow-up
|
|
2. **Run quality gates** (if code changed) - Tests, linters, builds
|
|
3. **Update issue status** - Close finished work, update in-progress items
|
|
4. **PUSH TO REMOTE** - This is MANDATORY:
|
|
```bash
|
|
git pull --rebase
|
|
bd dolt push
|
|
git push
|
|
git status # MUST show "up to date with origin"
|
|
```
|
|
5. **Clean up** - Clear stashes, prune remote branches
|
|
6. **Verify** - All changes committed AND pushed
|
|
7. **Hand off** - Provide context for next session
|
|
|
|
**CRITICAL RULES:**
|
|
- Work is NOT complete until `git push` succeeds
|
|
- NEVER stop before pushing - that leaves work stranded locally
|
|
- NEVER say "ready to push when you are" - YOU must push
|
|
- If push fails, resolve and retry until it succeeds
|
|
<!-- END BEADS INTEGRATION -->
|