docs(agents): restructure site docs and remove workflow references#28
docs(agents): restructure site docs and remove workflow references#28zrosenbauer wants to merge 22 commits intomainfrom
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: 63cb0d4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdded four new top-level documentation pages, renamed/rewired agent docs from “workflow” → “flowAgent” across examples and references, reorganized site navigation (including new Models section), tweaked package scripts/devDependency, adjusted vercel ignore logic, and added an empty changeset. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
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/quick-start.md`:
- Around line 6-23: The quick-start uses `@ai-sdk/openai` (import openai) but sets
OPENROUTER_API_KEY; fix by making auth and provider match: either change the
environment variable to OPENAI_API_KEY and document that consumers must export
OPENAI_API_KEY, or change the import/model to an OpenRouter-compatible provider
(and keep OPENROUTER_API_KEY). Update the doc lines that set the env var and/or
the import call (agent(...) using openai(...) / the OpenRouter provider symbol)
so the provider name and the env var (OPENAI_API_KEY or OPENROUTER_API_KEY) are
consistent.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 082133c8-6b34-4133-a91b-72bf10cd95f2
📒 Files selected for processing (23)
docs/architecture.mddocs/introduction.mddocs/principles.mddocs/quick-start.mdpackage.jsonpackages/agents/docs/advanced/streaming.mdpackages/agents/docs/core/flow-agent.mdpackages/agents/docs/core/hooks.mdpackages/agents/docs/core/overview.mdpackages/agents/docs/core/step.mdpackages/agents/docs/guides/cost-aware-agents.mdpackages/agents/docs/guides/create-agent.mdpackages/agents/docs/guides/create-flow-agent.mdpackages/agents/docs/guides/create-tool.mdpackages/agents/docs/guides/error-recovery.mdpackages/agents/docs/guides/multi-agent-orchestration.mdpackages/agents/docs/guides/test-agents.mdpackages/agents/docs/overview.mdpackages/agents/docs/provider/overview.mdpackages/agents/docs/provider/usage.mdpackages/agents/docs/troubleshooting.mdpackages/cli/README.mdzpress.config.ts
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
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/introduction.md`:
- Around line 55-59: Replace the ternary expressions that check result.ok and
docs.ok with explicit if/else assignments to follow project conventions: for the
expression returning result.ok ? result.value.output : "" replace it with an
if/else that checks result.ok and returns result.value.output when true and ""
when false (referencing the variable result and its .value.output), and for the
object return using docs.ok ? docs.value : [] replace it by assigning a local
variable (e.g., docsArray) via an if/else that sets docsArray = docs.value when
docs.ok is true and docsArray = [] otherwise, then return { docs: docsArray }
(references: the symbols result, result.value.output, docs, and docs.value).
In `@docs/principles.md`:
- Around line 113-118: Replace the ternary expressions in this example with
explicit control flow to match project guidelines: instead of using "const
cleaned = cleanText(analysis.ok ? analysis.value.output : input.text);" and
"return { result: final.ok ? final.value : cleaned }", rewrite using if/else (or
match()) so you first check analysis.ok and set cleaned from
cleanText(analysis.value.output) or cleanText(input.text), and later check
final.ok to return either final.value or cleaned; update the $.step call (id:
"format", execute: () => formatOutput(cleaned)) usage accordingly so the example
shows clear, idiomatic branching with the functions cleanText, formatOutput and
the variables analysis, input.text, cleaned, and final.
In `@docs/quick-start.md`:
- Line 114: Replace the ternary in the return expression so it follows project
conventions: instead of using "return { final: reviewed.ok ?
reviewed.value.output : draft.value.output }", assign final via an if/else (or a
match on reviewed.ok) using the reviewed and draft values (e.g., compute final =
reviewed.ok ? ... but implemented as if/else) and then return { final }; update
the example to use the reviewed, draft, and final identifiers so it clearly
shows the if/else (or match) pattern.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 1d1819fe-e3b3-4dc9-a249-a553ea2a146c
📒 Files selected for processing (7)
docs/architecture.mddocs/introduction.mddocs/principles.mddocs/quick-start.mdpackages/agents/docs/core/flow-agent.mdpackages/agents/docs/overview.mdzpress.config.ts
- Fix env var mismatch: OPENROUTER_API_KEY → OPENAI_API_KEY in quick-start - Replace ternaries with if/else in example code (introduction, principles, quick-start) Co-Authored-By: Claude <noreply@anthropic.com>
Replaces manual git diff path list with `pnpm zpress diff` which automatically derives source paths from zpress.config.ts. Co-Authored-By: Claude <noreply@anthropic.com>
- Migrate zpress.config.ts: link→path, from→include, prefix→path, titleFrom→title.from, packages→workspaces, remove isolated - Fix vercel.json ignoreCommand to include all doc source paths - Add .zpress-bugs.md documenting upstream issues (missing FileTree.css, zpress diff using git status) Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
All @zpress/ui 0.8.x versions have a broken FileTree.css import that prevents builds. Pin kit to 0.2.4 (uses ui 0.7.0) and restore the original zpress config schema (link/from/prefix/packages). Co-Authored-By: Claude <noreply@anthropic.com>
…chema Upgrade to latest @zpress/kit and migrate zpress.config.ts to the v0.4 config schema: link→path, from→include, prefix→path, titleFrom→title.from, packages→workspaces, isolated→standalone. Note: build is blocked by upstream @zpress/ui 0.8.x bug where rspressPluginFileTree references a missing FileTree.css file. Config validates successfully — build will work once upstream fix ships. Co-Authored-By: Claude <noreply@anthropic.com>
Picks up @zpress/ui 0.8.4 which fixes the missing FileTree.css bug. Co-Authored-By: Claude <noreply@anthropic.com>
Replace manual git diff with zpress diff --ref HEAD^ which automatically checks the configured source directories. Co-Authored-By: Claude <noreply@anthropic.com>
- Replace workspaces with features for homepage cards (with descriptions and mdi icons) - Switch all icons from pixelarticons to mdi for better rendering - Preserve directory-based path structure to avoid breaking internal relative links between doc files Co-Authored-By: Claude <noreply@anthropic.com>
- Add workspaces with single "Packages" category containing all 4 @funkai packages as cards with descriptions - Add hero action buttons for Quick Start and Introduction - Keep features for homepage feature cards Co-Authored-By: Claude <noreply@anthropic.com>
- Packages sections now point to README.md only (agents, models, prompts, cli) - All detailed docs moved under top-level Reference section, grouped by package - All guides moved under top-level Guides section, grouped by package - Fix broken relative links in READMEs to use absolute doc site paths Co-Authored-By: Claude <noreply@anthropic.com>
- Each package section (Agents, Models, Prompts, CLI) uses its README as the overview with all docs listed flat underneath - Remove Reference and Guides wrapper sections — everything lives directly under its package - Fix README links to use absolute doc site paths - Add back core/overview and sdk overview pages to resolve internal relative links Co-Authored-By: Claude <noreply@anthropic.com>
- Fix agent.generate()/stream() to use object params { prompt: "..." }
- Fix model field to use LanguageModel instances, not string IDs
- Rename workflow→flowAgent, createWorkflowEngine→createFlowEngine
- Rename createModelResolver→createProviderRegistry in models docs
- Remove phantom openrouter/createOpenRouter exports from agents docs
- Fix result.stream→result.fullStream in streaming examples
- Fix --roots→--includes CLI flag across all prompts docs
- Rewrite agents provider docs to reflect actual API
- Add @funkai/models to root README packages table
- Restructure zpress.config.ts with features, workspaces, sections
Co-Authored-By: Claude <noreply@anthropic.com>
… changelogs - Flatten agents docs from 29 nested files to 14 flat files (52% reduction) - Flatten models docs from 12 nested files to 5 flat files (58% reduction) - Flatten prompts docs from 12 nested files to 5 flat files (58% reduction) - Flatten CLI docs from 2 files to 1 file (50% reduction) - Move root docs (intro, quickstart, principles, architecture) into visible Getting Started sidebar section - Nest changelogs under each package in Packages section - Update feature card descriptions to be distinct from workspace descriptions - Update workspace item descriptions to short identifiers - Rewrite overview pages as concise landing pages with architecture diagrams - Fix all cross-links to match new flat file structure - Update models README references to new URL paths Co-Authored-By: Claude <noreply@anthropic.com>
Replace 4 generic package-name feature cards with 6 compelling value-prop cards. Fold the standalone CLI section (1 page) into the Prompts section to reduce navbar clutter. Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Updates workspaces → packages config field per breaking change. Co-Authored-By: Claude <noreply@anthropic.com>
- Flatten nav into Concepts / Reference / Guides sections - Update all internal links to new route structure - Fix inconsistent null handling in cost-tracking examples - Add abort signal propagation to multi-agent examples - Fix formatting issues caught by oxfmt Co-Authored-By: Claude <noreply@anthropic.com>
Summary
Test plan