Skip to content

docs: plan Forgejo provider implementation#265

Merged
mariusvniekerk merged 9 commits intomainfrom
forgejo-provider-impl
May 7, 2026
Merged

docs: plan Forgejo provider implementation#265
mariusvniekerk merged 9 commits intomainfrom
forgejo-provider-impl

Conversation

@mariusvniekerk
Copy link
Copy Markdown
Collaborator

docs: plan Forgejo provider implementation

Researches Forgejo API compatibility, token scopes, Go SDK options, and local fixture needs for adding Forgejo as the third Middleman provider after GitHub and GitLab.

docs: include Gitea in provider plan

Expands the Forgejo provider plan to add Gitea as a sibling provider, pin separate SDKs for each, and call out Forgejo-specific Actions behavior instead of treating both APIs as identical.

docs: share Forgejo and Gitea provider core

Reworks the provider plan so Forgejo and Gitea use a shared gitea-like implementation for common provider behavior, keeping the concrete provider packages focused on SDK adaptation and true API divergence such as Forgejo Actions.

docs: align Forgejo plan with GitHub provider features

Adds a GitHub capability parity target for Forgejo and Gitea, including read features, mutations, workflow approval, ready-for-review gaps, and tests that keep capability flags honest.

docs: resolve Forgejo provider design review

Clarifies the read-only MVP boundary, moves GitHub-parity mutations to post-MVP work, fixes SDK base URL guidance, adds required UI e2e coverage, and documents config regression and insufficient-scope error tests.

chore: close remaining Forgejo provider reviews

Records that the remaining roborev jobs for earlier Forgejo and Gitea provider-plan commits were resolved by the existing follow-up spec fixes and closed.

docs: require host-scoped forge tokens

Clarifies that Forgejo and Gitea token resolution is keyed by provider and host, with public-host defaults plus explicit platform token envs for federated and self-hosted instances.

docs: add Forgejo provider kata graph

Records the kata epic and implementation task graph for the Forgejo and Gitea provider work, including MVP sequencing and post-MVP parity tasks.

@roborev-ci
Copy link
Copy Markdown

roborev-ci Bot commented May 7, 2026

roborev: Combined Review (ba45b96)

No Medium, High, or Critical issues found.

All reviewers agree the diff is clean.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

Researches Forgejo API compatibility, token scopes, Go SDK options, and local fixture needs for adding Forgejo as the third Middleman provider after GitHub and GitLab.
Expands the Forgejo provider plan to add Gitea as a sibling provider, pin separate SDKs for each, and call out Forgejo-specific Actions behavior instead of treating both APIs as identical.
Reworks the provider plan so Forgejo and Gitea use a shared gitea-like implementation for common provider behavior, keeping the concrete provider packages focused on SDK adaptation and true API divergence such as Forgejo Actions.
Adds a GitHub capability parity target for Forgejo and Gitea, including read features, mutations, workflow approval, ready-for-review gaps, and tests that keep capability flags honest.
Clarifies the read-only MVP boundary, moves GitHub-parity mutations to post-MVP work, fixes SDK base URL guidance, adds required UI e2e coverage, and documents config regression and insufficient-scope error tests.
Records that the remaining roborev jobs for earlier Forgejo and Gitea provider-plan commits were resolved by the existing follow-up spec fixes and closed.
Clarifies that Forgejo and Gitea token resolution is keyed by provider and host, with public-host defaults plus explicit platform token envs for federated and self-hosted instances.
Records the kata epic and implementation task graph for the Forgejo and Gitea provider work, including MVP sequencing and post-MVP parity tasks.
@mariusvniekerk mariusvniekerk force-pushed the forgejo-provider-impl branch from ba45b96 to d62e56b Compare May 7, 2026 20:12
@roborev-ci
Copy link
Copy Markdown

roborev-ci Bot commented May 7, 2026

roborev: Combined Review (d62e56b)

No Medium, High, or Critical findings were reported.

All agents agree the diff is clean; no actionable review comments to post.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

The Playwright image pull happens before CI job steps can run, so our retry wrapper cannot help when MCR blocks or flakes. Revert the e2e jobs to install the requested browser on the hosted runner instead of running inside the Playwright container.
@roborev-ci
Copy link
Copy Markdown

roborev-ci Bot commented May 7, 2026

roborev: Combined Review (bb46292)

No Medium, High, or Critical issues found.

All reported reviews are clean; no actionable findings to include.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

@mariusvniekerk mariusvniekerk merged commit 294f6c4 into main May 7, 2026
9 of 17 checks passed
@mariusvniekerk mariusvniekerk deleted the forgejo-provider-impl branch May 7, 2026 23:19
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.

1 participant