feat: revise getting started experience with framework-first IA#274
Draft
chrisraygill wants to merge 2 commits into
Draft
feat: revise getting started experience with framework-first IA#274chrisraygill wants to merge 2 commits into
chrisraygill wants to merge 2 commits into
Conversation
Collaborator
chrisraygill
commented
May 9, 2026
- Add "Choose your stack" hub page as primary onboarding entry point
- Restructure sidebar: lean "Get started" section, new "Frameworks" section with language filtering, "Client SDKs" replaced by Web client and Flutter pages
- Add framework guides for Next.js App Router, Next.js Pages, Hono, Fastify, Gin, Echo, Chi, Django, and Web client
- Make Flutter guide language-agnostic (works with any Genkit backend)
- Add "Work with Genkit in AI tools" page for MCP/IDE setup
- Add CORS handling to Go framework guides (gin-contrib/cors, Echo middleware, go-chi/cors)
- Fix Output import bug in FastAPI and Flask guides (use output_schema= instead of Output())
- Ignore .scratch/ directory
Contributor
There was a problem hiding this comment.
Code Review
This pull request significantly restructures the Genkit documentation by introducing a "Choose your stack" entry point and adding or rewriting integration guides for various frameworks across JavaScript, Go, Python, and Dart. Key changes include new guides for Next.js (App and Pages routers), Fastify, Hono, Django, Gin, Echo, Chi, and a generic Web Client, alongside simplified versions of existing guides for Angular, Express, FastAPI, Flask, Flutter, and Shelf. Feedback focuses on correcting a recurring typo where gemini-2.5-flash was used instead of gemini-2.0-flash across multiple files, and improving request cancellation logic in the Fastify guide.
| headers, | ||
| body, | ||
| duplex: 'half', | ||
| signal: request.raw.aborted ? AbortSignal.abort() : undefined, |
Contributor
There was a problem hiding this comment.
Using AbortSignal.abort() only provides a signal that is already aborted. To correctly support request cancellation, you should pass the request's own signal.
signal: request.signal,
bed9dd6 to
7fcaf75
Compare
b86c88d to
441ff19
Compare
- Add "Choose your stack" hub page as primary onboarding entry point - Restructure sidebar: lean "Get started" section, new "Frameworks" section with language filtering, "Client SDKs" replaced by Web client and Flutter pages - Add framework guides for Next.js App Router, Next.js Pages, Hono, Fastify, Gin, Echo, Chi, Django, and Web client - Make Flutter guide language-agnostic (works with any Genkit backend) - Add "Work with Genkit in AI tools" page for MCP/IDE setup - Add CORS handling to Go framework guides (gin-contrib/cors, Echo middleware, go-chi/cors) - Fix Output import bug in FastAPI and Flask guides (use output_schema= instead of Output()) - Ignore .scratch/ directory
441ff19 to
2dfdda1
Compare
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.