Skip to content

Conversation

@ocodista
Copy link

@ocodista ocodista commented Nov 7, 2025

Summary

EdgeDB has rebranded to Gel. This PR comprehensively updates all documentation to reflect the rebrand.

Changes Made

Documentation & Branding

  • Guide title: "Use EdgeDB with Bun" → "Use Gel with Bun"
  • File renamed: docs/guides/ecosystem/edgedb.mdxgel.mdx
  • Description: Added "(formerly EdgeDB)" note
  • All path references: Updated from /guides/ecosystem/edgedb to /guides/ecosystem/gel

CLI Commands

  • edgedb project initgel project init
  • edgedbgel (REPL)
  • edgedb migration creategel migration create
  • edgedb migrategel migrate

npm Packages

  • edgedbgel
  • @edgedb/generate@gel/generate

Installation & Documentation URLs

  • Installation link: docs.geldata.com/learn/installation (functional)
  • Documentation reference: docs.geldata.com/ (operational)
  • Installation scripts: Verified working (https://www.geldata.com/sh and ps1)
  • Added Homebrew option: brew install geldata/tap/gel-cli

Code Examples

  • Updated all imports: import { createClient } from "gel"
  • Updated codegen commands: bunx @gel/generate

Verified

All commands verified against official Gel documentation at https://docs.geldata.com/

Fixes #17721

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 7, 2025

Walkthrough

Documentation and site metadata were updated to reflect a rebrand from EdgeDB to Gel: link text and guide paths changed, and the ecosystem guide content was fully rewritten to use Gel names, CLI, file references, imports, and package/tooling names.

Changes

Cohort / File(s) Summary
Navigation & index updates
README.md, docs/snippets/guides.jsx, docs/docs.json
Replaced Ecosystem link label and guide path: "Use EdgeDB with Bun" → "Use Gel with Bun"; updated guide hrefs from /guides/ecosystem/edgedb to /guides/ecosystem/gel
Ecosystem guide content
docs/guides/ecosystem/gel.mdx
Full content rewrite for rebrand: product name, CLI commands, config filenames (edgedb.tomlgel.toml), example project names, client imports and usage (edgedbgel), codegen/tooling package names (@edgedb/generate@gel/generate), and updated example commands and docs links

Pre-merge checks

✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating all EdgeDB references to Gel rebrand throughout the documentation.
Description check ✅ Passed The description provides comprehensive details about the rebrand changes, including specific file updates, CLI commands, npm packages, and verification steps, though it deviates from the template structure.
Linked Issues check ✅ Passed All coding requirements from issue #17721 are met: EdgeDB references replaced with Gel equivalents across documentation, CLI commands, npm packages, URLs, and code examples.
Out of Scope Changes check ✅ Passed All changes are directly related to the EdgeDB-to-Gel rebrand objective; no out-of-scope modifications detected in the files altered.

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between f88cd07 and 269db0f.

📒 Files selected for processing (1)
  • docs/docs.json (1 hunks)
🔇 Additional comments (1)
docs/docs.json (1)

357-357: ✓ Path correctly updated to reflect Gel rebrand.

The ecosystem guide path is correctly updated from /guides/ecosystem/edgedb to /guides/ecosystem/gel, aligning with the file rename and the broader rebrand documented in this PR.


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

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0db90b2 and 6ada2f4.

📒 Files selected for processing (1)
  • docs/guides/ecosystem/edgedb.mdx (2 hunks)

Update all EdgeDB documentation to reflect rebrand to Gel:
- Update guide title from "Use EdgeDB with Bun" to "Use Gel with Bun"
- Update CLI commands (edgedb → gel)
- Update npm packages (edgedb → gel, @edgedb/generate → @gel/generate)
- Update installation URLs and scripts
- Update all code examples and imports
- Add "(formerly EdgeDB)" note to description

Fixes oven-sh#17721
@ocodista ocodista force-pushed the claude/edgedb-rebrand branch from 6ada2f4 to e75babc Compare November 7, 2025 20:32
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

♻️ Duplicate comments (1)
docs/snippets/guides.jsx (1)

126-126: Inconsistency between title and URL path (duplicate).

Same issue as in README.md: The title displays "Use Gel with Bun" but href remains /guides/ecosystem/edgedb. This creates a mismatch between branding and navigation paths.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 6ada2f4 and e75babc.

📒 Files selected for processing (3)
  • README.md (1 hunks)
  • docs/guides/ecosystem/edgedb.mdx (9 hunks)
  • docs/snippets/guides.jsx (1 hunks)
🧰 Additional context used
🧠 Learnings (6)
📚 Learning: 2025-08-30T00:12:56.803Z
Learnt from: CR
Repo: oven-sh/bun PR: 0
File: .cursor/rules/writing-tests.mdc:0-0
Timestamp: 2025-08-30T00:12:56.803Z
Learning: Applies to test/**/*.{js,ts} : Write tests in JavaScript or TypeScript using Bun’s Jest-style APIs (test, describe, expect) and run with bun test

Applied to files:

  • README.md
📚 Learning: 2025-10-19T02:44:46.354Z
Learnt from: theshadow27
Repo: oven-sh/bun PR: 23798
File: packages/bun-otel/context-propagation.test.ts:1-1
Timestamp: 2025-10-19T02:44:46.354Z
Learning: In the Bun repository, standalone packages under packages/ (e.g., bun-vscode, bun-inspector-protocol, bun-plugin-yaml, bun-plugin-svelte, bun-debug-adapter-protocol, bun-otel) co-locate their tests with package source code using *.test.ts files. This follows standard npm/monorepo patterns. The test/ directory hierarchy (test/js/bun/, test/cli/, test/js/node/) is reserved for testing Bun's core runtime APIs and built-in functionality, not standalone packages.

Applied to files:

  • README.md
📚 Learning: 2025-10-15T04:01:16.478Z
Learnt from: AmanVarshney01
Repo: oven-sh/bun PR: 23580
File: docs/guides/ecosystem/prisma.md:21-26
Timestamp: 2025-10-15T04:01:16.478Z
Learning: For Prisma with Bun runtime and Rust-free client (engineType "client"), when using local SQLite files (file:// URLs), use either prisma/adapter-better-sqlite3 or the Bun-native synapsenwerkstatt/prisma-bun-sqlite-adapter. The prisma/adapter-libsql adapter is only for remote libSQL/Turso endpoints and will not work with local SQLite file URLs.

Applied to files:

  • README.md
📚 Learning: 2025-10-04T09:52:49.414Z
Learnt from: CR
Repo: oven-sh/bun PR: 0
File: src/js/CLAUDE.md:0-0
Timestamp: 2025-10-04T09:52:49.414Z
Learning: Applies to src/js/bun/**/*.{ts,js} : Place Bun-specific modules (e.g., bun:ffi, bun:sqlite) under bun/

Applied to files:

  • README.md
📚 Learning: 2025-10-19T02:52:37.412Z
Learnt from: theshadow27
Repo: oven-sh/bun PR: 23798
File: packages/bun-otel/tsconfig.json:1-15
Timestamp: 2025-10-19T02:52:37.412Z
Learning: In the Bun repository, packages under packages/ (e.g., bun-otel) can follow a TypeScript-first pattern where package.json exports point directly to .ts files (not compiled .js files). Bun natively runs TypeScript, so consumers import .ts sources directly and receive full type information without needing compiled .d.ts declaration files. For such packages, adding "declaration": true or "outDir" in tsconfig.json is unnecessary and would break the export structure.
<!-- [remove_learning]
ceedde95-980e-4898-a2c6-40ff73913664

Applied to files:

  • README.md
📚 Learning: 2025-09-02T05:33:37.517Z
Learnt from: Jarred-Sumner
Repo: oven-sh/bun PR: 22323
File: test/js/web/websocket/websocket-subprotocol.test.ts:74-75
Timestamp: 2025-09-02T05:33:37.517Z
Learning: In Bun's runtime, `await using` with Node.js APIs like `net.createServer()` is properly supported and should not be replaced with explicit cleanup. Bun has extended Node.js APIs with proper async dispose support.

Applied to files:

  • README.md
🔇 Additional comments (2)
docs/guides/ecosystem/edgedb.mdx (2)

135-136: Verified: npm packages exist and are published.

Both package names referenced in the guide are valid and available on npm:

  • gel: v2.1.1
  • @gel/generate: v0.7.0

The installation commands will not fail due to package unavailability.

Likely an incorrect or invalid review comment.


40-40: All review concerns have been verified and found to be unfounded.

The guide is accurate:

  1. The gel CLI is officially distributed via https://www.geldata.com/sh and https://www.geldata.com/ps1, which matches the installation scripts shown in the guide.

  2. The command name has been officially changed from edgedb to gel, as confirmed by the guide's opening statement: "Gel (formerly EdgeDB)".

  3. All commands documented in the guide (gel project init, gel migration create, and gel migrate) are officially supported. The guide correctly reflects the current gel CLI API.

No changes needed.

Likely an incorrect or invalid review comment.

@ocodista ocodista marked this pull request as draft November 7, 2025 20:39
Replace broken geldata.com URLs with functional docs.geldata.com endpoints:
- Update install link to docs.geldata.com/learn/installation (was 404)
- Update docs reference to docs.geldata.com (was broken redirect)
- Add Homebrew installation method for additional options

Addresses CodeRabbit review feedback about non-functional URLs.
@ocodista ocodista changed the title Update EdgeDB documentation URLs to GelData docs: update EdgeDB references to Gel rebrand Nov 7, 2025
Rename guide file from edgedb.mdx to gel.mdx and update all references:
- Rename docs/guides/ecosystem/edgedb.mdx to gel.mdx
- Update path references in README.md, docs.json, and guides.jsx
- Maintain consistency with Gel rebrand throughout documentation

This ensures URL paths match the new Gel branding.
@ocodista ocodista marked this pull request as ready for review November 7, 2025 20:49
@RiskyMH RiskyMH added the docs Improvements or additions to documentation label Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EdgeDB Rebrand

3 participants