Add "Who We Are" page to docs about section#118
Merged
SantiagoDePolonia merged 17 commits intomainfrom Mar 6, 2026
Merged
Conversation
Introduces the people and organization behind GOModel, explains the relationship with Enterpilot, and addresses open-source licensing intentions with references to MongoDB/Elastic/Redis precedents. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Enterpilot is a full AI access and traffic management platform (closer to Dust.tt), not just GOModel with enterprise features on top. Updated to reflect chat interface, API interface, security/PII tooling, cost management, observability, and deployment options. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Contributor
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds a new documentation page Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/about/who-we-are.mdx`:
- Around line 27-28: Replace the redundant phrase "API interface" with a concise
alternative (e.g., "API" or "API endpoint") in the string "**API interface** — a
single API endpoint that routes to any supported model, with per-user and
per-team budget controls." so the sentence reads clearly (for example: "**API
endpoint** — a single API endpoint that routes to any supported model, with
per-user and per-team budget controls.").
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: fe1554de-4895-4d8d-a79b-82f2a0c6de5a
📒 Files selected for processing (2)
docs/about/who-we-are.mdxdocs/docs.json
Remove API interface, security, cost management, observability, and deployment flexibility bullets. Add internal company knowledge base integration description instead. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Point to enterpilot.io and the GitHub README, mention Discord invite link is available there. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Keep it concise: open source, needs to be sustainable, narrow license changes only if necessary. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Add back MongoDB, Elastic, and Redis license change examples to provide context for the licensing stance. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Knowledge base integration is not yet available — move it from the current Enterpilot features list to the upcoming features paragraph. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Adds context on why infrastructure should be open source, then clearly separates the two scenarios where the license might change: protecting the project from cloud provider extraction, and long-term sustainability. https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Contributor
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/about/who-we-are.mdx`:
- Around line 30-43: Merge the repeated messaging in the "Why open source" and
"On licensing" sections by combining their duplicate sentences about
infrastructure, auditability, and extensibility into one concise paragraph; keep
a single section header (either "Why open source" or "On licensing") and remove
the redundant lines so the text still highlights that GOModel is open source to
allow reading, auditing, extending, and community contribution without repeating
the same phrases across both section openings; update the content under the
retained header to flow logically (introduce the infrastructure rationale then
state licensing/open-source commitment) and delete the copied sentences in the
other section.
- Around line 52-58: The Elastic blog URL in the list ("Elastic moved to a dual
license" currently linking to
https://www.elastic.co/blog/why-license-change-was-necessary) is returning 404;
update that list item in who-we-are.mdx by replacing the broken URL with a
working source (e.g., an archived copy or an official Elastic post on the
license change) or remove the hyperlink and/or entire bullet if no reliable
replacement exists, ensuring the visible link text "Elastic moved to a dual
license" remains accurate or is adjusted to reflect the new source.
- Around line 23-28: The "Upcoming features..." paragraph is orphaned and breaks
the list under the "**Chat interface** —" bullet; fix it by making it a proper
list item or a continuation: either indent the "Upcoming features include..."
lines as a continuation of the "**Chat interface** —" bullet (indent the lines
to match the list item so they render as part of that bullet), or convert the
paragraph into its own bullet (e.g., prepend "- Upcoming features include..."
and wrap the subsequent feature list lines accordingly) so the markdown renders
as intended.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 942aca75-a1ff-4814-bbf0-87ff24d83ddc
📒 Files selected for processing (1)
docs/about/who-we-are.mdx
Comment on lines
+30
to
+43
| ## Why open source | ||
|
|
||
| GOModel is open source because an AI gateway is infrastructure. Infrastructure | ||
| works best when people can read it, audit it, and extend it. | ||
|
|
||
| We want the community to use GOModel freely, contribute improvements, and trust | ||
| that the project will keep shipping. | ||
|
|
||
| ## On licensing | ||
|
|
||
| An AI gateway is infrastructure. It sits between every request your application | ||
| makes and every model it talks to. That kind of component should be something | ||
| you can read, audit, extend, and trust — which is why GOModel is open source | ||
| today, with no asterisks. |
Contributor
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
Consider consolidating redundant content.
The "Why open source" section (lines 30-36) and the opening of "On licensing" (lines 40-43) repeat similar points about infrastructure, auditing, and extending. Consider merging these into a single cohesive section to avoid repetition.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/about/who-we-are.mdx` around lines 30 - 43, Merge the repeated messaging
in the "Why open source" and "On licensing" sections by combining their
duplicate sentences about infrastructure, auditability, and extensibility into
one concise paragraph; keep a single section header (either "Why open source" or
"On licensing") and remove the redundant lines so the text still highlights that
GOModel is open source to allow reading, auditing, extending, and community
contribution without repeating the same phrases across both section openings;
update the content under the retained header to flow logically (introduce the
infrastructure rationale then state licensing/open-source commitment) and delete
the copied sentences in the other section.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
The old URL (why-license-change-was-necessary) returns 404. Replaced with the actual licensing change announcement post (licensing-change). https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Introduces the people and organization behind GOModel, explains the relationship with Enterpilot, and addresses open-source licensing intentions with references to MongoDB/Elastic/Redis precedents.
https://claude.ai/code/session_01QKDe5a73tpRkmsouevuCNs
Summary by CodeRabbit