Skip to content

docs(direction): commit forge-agnostic support as a maintained direction#1575

Merged
Wirasm merged 1 commit into
devfrom
docs/direction-forge-agnostic
May 4, 2026
Merged

docs(direction): commit forge-agnostic support as a maintained direction#1575
Wirasm merged 1 commit into
devfrom
docs/direction-forge-agnostic

Conversation

@Wirasm

@Wirasm Wirasm commented May 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Why now

@truck0321's PR #1104 has been waiting on a maintainer direction call. The implementation is well-tested and clean; the only outstanding question was whether forge support beyond GitHub is in scope. The answer is yes — but as community-tier (matching how Pi is positioned vs. Claude/Codex), not first-class core support.

Filing this as a separate one-line PR so the direction call is its own auditable change, decoupled from any code that follows it.

Test plan

  • No code changes — pure docs.
  • CI green (lint/format on the YAML/MD files only).
  • Confirm `bun run validate` clean.

Summary by CodeRabbit

  • Documentation
    • Clarified that Archon is forge-agnostic, with GitHub as the primary platform and Gitea and GitLab supported via community adapters for PR, issue, and review operations.

GitHub stays the primary forge; Gitea and GitLab are community-supported
targets via the existing community adapters at
packages/adapters/src/community/forge/. Long-term home for outbound forge
operations (PR/issue/review CRUD) is the same per-forge adapter that
already handles inbound webhooks today.

Closes the open direction question that has been blocking PR #1104
(forge-agnostic workflow commands) for ~8 days.

The architectural cleanup — migrating PR #1104's interim forge-cli.ts
into the existing community adapters — is tracked separately as #1574
and is not a precondition for merging #1104.
@coderabbitai

coderabbitai Bot commented May 4, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This PR adds a single clarifying bullet point to the maintainer direction document specifying that Archon is forge-agnostic, with GitHub as the primary forge and Gitea/GitLab supported via community adapters, with a shared per-forge adapter handling both inbound webhooks and outbound PR/issue/review operations.

Changes

Documentation Update

Layer / File(s) Summary
Direction Documentation
.archon/maintainer-standup/direction.md
Added a "What Archon IS" bullet clarifying forge-agnostic architecture with GitHub as primary, Gitea/GitLab support via community adapters, and shared adapter responsibility for inbound/outbound operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A forge is just a path, they say,
GitHub leads the way, hooray!
But Gitea and GitLab too,
Dance with Archon, shiny new.
One adapter, clever friend,
Webhooks in, and out they send! 🚀

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing critical sections from the template including UX Journey, Architecture Diagram, Label Snapshot, Change Metadata, Validation Evidence, Security Impact, Compatibility, Human Verification, Side Effects, Rollback Plan, and Risks sections. Complete the description template by adding all required sections. For documentation-only changes, adapt sections appropriately (e.g., note N/A for architecture/validation where not applicable).
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding a forge-agnostic direction to the documentation, specifying GitHub as primary and community support for Gitea/GitLab.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/direction-forge-agnostic

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
.archon/maintainer-standup/direction.md (1)

18-18: ⚡ Quick win

Tighten this clause for scanability and fix hyphenation.

Line 18 is directionally correct, but it’s doing a lot in one bullet; consider tightening it to match the quick-scan intent of this doc, and change “community supported” to “community-supported.”

✍️ Suggested rewrite
-- **Forge-agnostic.** GitHub is the primary forge, but Gitea and GitLab are community supported targets via community adapters at `packages/adapters/src/community/forge/`. Long-term home for outbound forge operations (PR/issue/review CRUD) is the same per-forge adapter that handles inbound webhooks. New forges land as new community adapters that implement the shared interface.
+- **Forge-agnostic.** GitHub is the primary forge. Gitea and GitLab are community-supported targets via adapters in `packages/adapters/src/community/forge/`. Outbound forge operations (PR/issue/review CRUD) live in the same per-forge adapter that handles inbound webhooks; new forges should land as community adapters implementing the shared interface.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.archon/maintainer-standup/direction.md at line 18, Tighten and hyphenate
the "Forge-agnostic." bullet: rewrite the single long sentence into a concise,
scan-friendly phrase that uses "community-supported" (hyphenated) and keeps the
essential points — GitHub primary, Gitea/GitLab supported via community
adapters, and the long-term home for outbound forge operations is the same
per-forge adapter that handles inbound webhooks — while retaining the reference
to packages/adapters/src/community/forge/ so readers can find the adapters path.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.archon/maintainer-standup/direction.md:
- Line 18: Tighten and hyphenate the "Forge-agnostic." bullet: rewrite the
single long sentence into a concise, scan-friendly phrase that uses
"community-supported" (hyphenated) and keeps the essential points — GitHub
primary, Gitea/GitLab supported via community adapters, and the long-term home
for outbound forge operations is the same per-forge adapter that handles inbound
webhooks — while retaining the reference to
packages/adapters/src/community/forge/ so readers can find the adapters path.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 31aa3d5c-5ba3-4460-98c7-8903ce3a682a

📥 Commits

Reviewing files that changed from the base of the PR and between 0c5d7b1 and 8ce42e3.

📒 Files selected for processing (1)
  • .archon/maintainer-standup/direction.md

@Wirasm Wirasm merged commit 41c0f17 into dev May 4, 2026
3 of 4 checks passed
@Wirasm Wirasm deleted the docs/direction-forge-agnostic branch May 4, 2026 16:43
@coderabbitai coderabbitai Bot mentioned this pull request May 5, 2026
cropse pushed a commit to cropse/Archon that referenced this pull request May 19, 2026
…ion (coleam00#1575)

GitHub stays the primary forge; Gitea and GitLab are community-supported
targets via the existing community adapters at
packages/adapters/src/community/forge/. Long-term home for outbound forge
operations (PR/issue/review CRUD) is the same per-forge adapter that
already handles inbound webhooks today.

Closes the open direction question that has been blocking PR coleam00#1104
(forge-agnostic workflow commands) for ~8 days.

The architectural cleanup — migrating PR coleam00#1104's interim forge-cli.ts
into the existing community adapters — is tracked separately as coleam00#1574
and is not a precondition for merging coleam00#1104.
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