fix: split testnet faucet into Asimov and Bradbury#388
Merged
MuncleUscles merged 16 commits intomainfrom Apr 7, 2026
Merged
Conversation
✅ Deploy Preview for genlayer-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 24 minutes and 25 seconds. ⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughReorganizes and consolidates protocol docs into a new canonical "What is GenLayer" page with redirects for deprecated pages, revises Optimistic Democracy and transaction lifecycle content, adds/updates Intelligent Contract feature pages (including image-processing and value-transfers), updates homepage messaging, and adds Testnet Bradbury to developer environment docs. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 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 |
…, update careers link
New: - what-is-genlayer: unified page with current messaging, architecture, comparison table, code example Rewritten: - how-genlayer-works: correct transaction lifecycle (including Leader Revealing stage), updated non-determinism framing Updated: - typical-use-cases: added Autonomous Agents - _meta.json: clean 4-entry nav Redirected: - what-are-intelligent-contracts → what-is-genlayer - what-makes-genlayer-different → what-is-genlayer - why-we-are-building-genlayer → what-is-genlayer - who-is-genlayer-for → what-is-genlayer
…cation primitive, portal link
MuncleUscles
commented
Apr 7, 2026
Member
Author
MuncleUscles
left a comment
MuncleUscles
left a comment
There was a problem hiding this comment.
We can also handle images, I think we should try an promote this as more of a highlight
| <Card title="😵💫 Types" href="../types" /> | ||
| <Card title="🔢 Types" href="../types" /> | ||
| <Card title="💾 Storage" href="storage" /> | ||
| <Card title="🐛 Error Handling" href="error-handling" /> |
Member
Author
There was a problem hiding this comment.
icon doesn't make sense
| <Card title="💸 Value Transfers" href="value-transfers" /> | ||
| <Card title="📨 Messages" href="messages" /> | ||
| <Card title="🧠 Interacting with Intelligent Contracts" href="interacting-with-intelligent-contracts" /> | ||
| <Card title="🐒 Interacting with EVM Contracts" href="interacting-with-evm-contracts" /> |
Member
Author
There was a problem hiding this comment.
icon doesn't make sense
|
|
||
| | Feature | Traditional Smart Contracts | Intelligent Contracts | | ||
| |---|---|---| | ||
| | **Language** | Solidity, Vyper | Python | |
Member
Author
There was a problem hiding this comment.
idk how popular vyper is, rust maybe for solana?
|
|
||
| **GenLayer Chain** — an EVM-compatible L2 (zkSync Elastic Chain). Holds account balances via ghost contracts, handles standard Ethereum operations (`eth_*` methods), and anchors to Ethereum's security model. | ||
|
|
||
| **GenVM** — the execution environment for Intelligent Contracts. A Python runtime with native access to LLMs, web data, and non-deterministic operations. |
Member
Author
There was a problem hiding this comment.
genmv is not just python - it's wasm, with a python interpreter, but can also run native compiled code. check repo to read more
|
|
||
| @gl.public.write | ||
| def resolve(self) -> None: | ||
| result = gl.eq_principle.call_llm( |
Member
Author
There was a problem hiding this comment.
This is not valid code, check the docs properly
…, code example - Error Handling: 🐛→⚠️ , EVM Contracts: 🐒→🔗 - Comparison table: Vyper→Rust (Solana) - GenVM: WebAssembly-based VM (Wasmtime) with Python interpreter + native code - Code example: replaced invalid code with WizardOfCoin from actual docs
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
pages/understand-genlayer-protocol/who-is-genlayer-for.mdx (1)
1-5: UnusedCalloutimport.The
Calloutcomponent is imported but not used in this redirect stub. Consider either removing the import or wrapping the redirect notice in aCalloutfor better visual prominence.Option 1: Remove unused import
-import { Callout } from "nextra-theme-docs"; - # Who Is GenLayer For? This page has moved to [What is GenLayer](/understand-genlayer-protocol/what-is-genlayer).Option 2: Use Callout for better visibility
import { Callout } from "nextra-theme-docs"; # Who Is GenLayer For? -This page has moved to [What is GenLayer](/understand-genlayer-protocol/what-is-genlayer). +<Callout type="info"> + This page has moved to [What is GenLayer](/understand-genlayer-protocol/what-is-genlayer). +</Callout>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pages/understand-genlayer-protocol/who-is-genlayer-for.mdx` around lines 1 - 5, Remove the unused Callout import or use it to wrap the redirect message: either delete the import statement "import { Callout } from \"nextra-theme-docs\";" to eliminate the unused symbol, or replace the plain markdown notice with a JSX Callout component (e.g., <Callout>...This page has moved to <a href="/understand-genlayer-protocol/what-is-genlayer">What is GenLayer</a>...</Callout>) so the imported Callout symbol is actually used.pages/understand-genlayer-protocol/typical-use-cases.mdx (1)
17-20: Consider varying the repeated “Does …” bullet openings.The four consecutive bullets scan a bit repetitive; light rewording would improve readability.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pages/understand-genlayer-protocol/typical-use-cases.mdx` around lines 17 - 20, The four consecutive bullet lines starting with "Does a new prediction market..." create repetitive openings; replace some "Does ..." leads with varied phrasing to improve flow — e.g., reword one or two to active or descriptive starts like "Is a new prediction market eligible for listing?", "Assess whether a DAO proposal complies with the organization's charter", "Ensure a content submission follows community standards", "Verify a transaction complies with regulatory requirements" — update the bullet texts in the section containing those four items to mix question and imperative phrasing while preserving original meaning.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pages/index.mdx`:
- Line 20: The grid uses a fixed 3-column value in the inline style
(gridTemplateColumns: 'repeat(3, minmax(0, 1fr))'), which breaks on smaller
viewports; update the style on that div to a responsive pattern such as using
repeat(auto-fit or auto-fill) with minmax (e.g., minmax(240px, 1fr)) or replace
the inline style with responsive CSS classes/media queries so the container (the
div with gridTemplateColumns) automatically collapses to fewer columns on narrow
screens.
---
Nitpick comments:
In `@pages/understand-genlayer-protocol/typical-use-cases.mdx`:
- Around line 17-20: The four consecutive bullet lines starting with "Does a new
prediction market..." create repetitive openings; replace some "Does ..." leads
with varied phrasing to improve flow — e.g., reword one or two to active or
descriptive starts like "Is a new prediction market eligible for listing?",
"Assess whether a DAO proposal complies with the organization's charter",
"Ensure a content submission follows community standards", "Verify a transaction
complies with regulatory requirements" — update the bullet texts in the section
containing those four items to mix question and imperative phrasing while
preserving original meaning.
In `@pages/understand-genlayer-protocol/who-is-genlayer-for.mdx`:
- Around line 1-5: Remove the unused Callout import or use it to wrap the
redirect message: either delete the import statement "import { Callout } from
\"nextra-theme-docs\";" to eliminate the unused symbol, or replace the plain
markdown notice with a JSX Callout component (e.g., <Callout>...This page has
moved to <a href="/understand-genlayer-protocol/what-is-genlayer">What is
GenLayer</a>...</Callout>) so the imported Callout symbol is actually used.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: c39f4509-f84b-49aa-bc22-0dae03981372
📒 Files selected for processing (12)
pages/developers/intelligent-contracts/features/features.mdxpages/developers/intelligent-contracts/features/value-transfers.mdxpages/developers/intelligent-contracts/introduction.mdxpages/index.mdxpages/understand-genlayer-protocol/_meta.jsonpages/understand-genlayer-protocol/optimistic-democracy-how-genlayer-works.mdxpages/understand-genlayer-protocol/typical-use-cases.mdxpages/understand-genlayer-protocol/what-are-intelligent-contracts.mdxpages/understand-genlayer-protocol/what-is-genlayer.mdxpages/understand-genlayer-protocol/what-makes-genlayer-different.mdxpages/understand-genlayer-protocol/who-is-genlayer-for.mdxpages/understand-genlayer-protocol/why-we-are-building-genlayer.mdx
|
|
||
| <br/> | ||
| <div style={{ display: 'grid', gridTemplateColumns: 'repeat(2, minmax(0, 1fr))', gap: '1rem' }}> | ||
| <div style={{ display: 'grid', gridTemplateColumns: 'repeat(3, minmax(0, 1fr))', gap: '1rem' }}> |
Contributor
There was a problem hiding this comment.
Make the card grid responsive instead of fixed at 3 columns.
A hardcoded 3-column layout can make cards too narrow on smaller screens.
Proposed fix
-<div style={{ display: 'grid', gridTemplateColumns: 'repeat(3, minmax(0, 1fr))', gap: '1rem' }}>
+<div style={{ display: 'grid', gridTemplateColumns: 'repeat(auto-fit, minmax(220px, 1fr))', gap: '1rem' }}>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| <div style={{ display: 'grid', gridTemplateColumns: 'repeat(3, minmax(0, 1fr))', gap: '1rem' }}> | |
| <div style={{ display: 'grid', gridTemplateColumns: 'repeat(auto-fit, minmax(220px, 1fr))', gap: '1rem' }}> |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@pages/index.mdx` at line 20, The grid uses a fixed 3-column value in the
inline style (gridTemplateColumns: 'repeat(3, minmax(0, 1fr))'), which breaks on
smaller viewports; update the style on that div to a responsive pattern such as
using repeat(auto-fit or auto-fill) with minmax (e.g., minmax(240px, 1fr)) or
replace the inline style with responsive CSS classes/media queries so the
container (the div with gridTemplateColumns) automatically collapses to fewer
columns on narrow screens.
- Move features index to sibling features.mdx so Nextra makes the sidebar folder title clickable (like core-concepts) - New Image Processing page: exec_prompt with images, web screenshots, use cases (visual evidence, documents, web monitoring) - Add Image Processing card to features grid
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
pages/understand-genlayer-protocol/typical-use-cases.mdx (1)
17-20: Reduce repeated sentence starts in the criteria checklist.Line 17 to Line 20 repeat “Does …” in each bullet. Consider slight variation for smoother reading.
✍️ Optional wording refinement
-- Does a new prediction market meet listing guidelines? -- Does a DAO proposal comply with the organization's charter? -- Does a content submission follow community standards? -- Does a transaction comply with regulatory requirements? +- Whether a new prediction market meets listing guidelines +- Whether a DAO proposal complies with the organization's charter +- Whether a content submission follows community standards +- Whether a transaction complies with regulatory requirements🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pages/understand-genlayer-protocol/typical-use-cases.mdx` around lines 17 - 20, The four checklist bullets starting with "Does ..." (the items about a new prediction market, a DAO proposal, a content submission, and a transaction) repeat the same sentence start; update them to vary phrasing for smoother reading by replacing some "Does ..." starts with alternatives like "Whether ...", "Assess whether ...", "Confirm that ...", or "Verify that ..." so the content remains the same but the sentence openings differ and flow better.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@pages/understand-genlayer-protocol/typical-use-cases.mdx`:
- Around line 17-20: The four checklist bullets starting with "Does ..." (the
items about a new prediction market, a DAO proposal, a content submission, and a
transaction) repeat the same sentence start; update them to vary phrasing for
smoother reading by replacing some "Does ..." starts with alternatives like
"Whether ...", "Assess whether ...", "Confirm that ...", or "Verify that ..." so
the content remains the same but the sentence openings differ and flow better.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: c279b461-2882-4e54-900e-226a748abde1
📒 Files selected for processing (7)
pages/developers/intelligent-contracts/features.mdxpages/developers/intelligent-contracts/features/_meta.jsonpages/developers/intelligent-contracts/features/features.mdxpages/developers/intelligent-contracts/features/image-processing.mdxpages/developers/intelligent-contracts/first-contract.mdxpages/understand-genlayer-protocol/typical-use-cases.mdxpages/understand-genlayer-protocol/what-is-genlayer.mdx
✅ Files skipped from review due to trivial changes (5)
- pages/developers/intelligent-contracts/features/_meta.json
- pages/developers/intelligent-contracts/first-contract.mdx
- pages/developers/intelligent-contracts/features/image-processing.mdx
- pages/understand-genlayer-protocol/what-is-genlayer.mdx
- pages/developers/intelligent-contracts/features.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
- pages/developers/intelligent-contracts/features/features.mdx
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
1 participant
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.
Split single Testnet row into Testnet Asimov and Testnet Bradbury. Added link to Networks page.
Summary by CodeRabbit