Skip to content

feat: add GET /repos/:did/:repo/pulls/:n/files endpoint#158

Merged
LiranCohen merged 1 commit intomainfrom
feat/pr-files
Mar 2, 2026
Merged

feat: add GET /repos/:did/:repo/pulls/:n/files endpoint#158
LiranCohen merged 1 commit intomainfrom
feat/pr-files

Conversation

@LiranCohen
Copy link
Contributor

Summary

Closes #123.

  • Adds GET /repos/:did/:repo/pulls/:number/files endpoint to the GitHub shim router and handler.
  • Returns changed files for a pull request. Since DWN revision records store only aggregate diff stats (additions, deletions, files changed), the response returns a summary entry with the totals rather than per-file details.
  • Rejects non-GET methods with 405.
  • Includes 2 new hardening tests (route matching and method validation).
  • Updated server startup banner and module docs.

Build, lint, and all 54 hardening tests pass.

Returns changed files for a pull request using aggregate diff stats from
DWN revision records. Per-file breakdown is not yet available — returns a
summary entry with totals.

Closes #123
@LiranCohen LiranCohen merged commit 13c9954 into main Mar 2, 2026
4 checks passed
@LiranCohen LiranCohen deleted the feat/pr-files branch March 2, 2026 05:21
@github-actions github-actions bot mentioned this pull request Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub shim: missing GET /repos/:did/:repo/pulls/:n/files endpoint

1 participant