docs: add "One Card per Step" Studio page, nested under Crew Studio (AGE-107)#6317
Conversation
![corridor-security[bot]](https://avatars.githubusercontent.com/in/1177339?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary: This PR adds static documentation pages and navigation entries for Studio behavior. No exploitable security vulnerabilities were identified.
Risk: Low risk. The changes are content/navigation-only and do not introduce executable code, user input handling, authentication/authorization logic, or external integrations.
📝 WalkthroughWalkthroughUpdates Crew Studio documentation navigation to group ChangesCrew Studio documentation
Suggested Reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
docs/v1.14.7/en/enterprise/features/merged-step-card.mdx (1)
1-83: 📐 Maintainability & Code Quality | 🟠 Major | 🏗️ Heavy liftDo not add pages under frozen
docs/v*snapshots in this PR.Line 1 places this new page under
docs/v1.14.7/..., which violates the repo freeze policy unless this is a[docs-freeze]release-cut PR.As per coding guidelines, "Never modify files under
docs/v*/as they are frozen release snapshots, except in[docs-freeze]release-cut PRs".🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/v1.14.7/en/enterprise/features/merged-step-card.mdx` around lines 1 - 83, This page was added under a frozen docs snapshot, which is not allowed in a normal PR. Remove the new `merged-step-card.mdx` content from the `docs/v1.14.7` snapshot path and place it in the appropriate non-frozen docs source location instead; if this is a release-cut change, only keep it in the frozen snapshot when the PR is explicitly marked `[docs-freeze]`. Use the `merged-step-card.mdx` document itself as the target to relocate, preserving its content outside `docs/v*`.Source: Coding guidelines
docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx (1)
1-83: 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick winDo not add new docs in frozen
docs/v*snapshots outside docs-freeze PRs.Lines 1-83 add a new page under
docs/v1.14.7/..., which conflicts with the repository freeze rule for versioned docs. Please move this content to edge-only (or run it via an explicit[docs-freeze]release-cut PR).As per coding guidelines: "
docs/v[0-9]*.[0-9]*.[0-9]*/**: Never modify files underdocs/v*/as they are frozen release snapshots, except in[docs-freeze]release-cut PRs."🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx` around lines 1 - 83, The new Arabic page is being added under a frozen versioned docs snapshot, which is not allowed outside a docs-freeze release-cut. Move this content out of the `docs/v1.14.7` snapshot into the edge-only docs location, or make the change only in a `[docs-freeze]` PR; use the `merged-step-card.mdx` page content and its related assets/sections as the items to relocate.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx`:
- Around line 1-83: The new Arabic page is being added under a frozen versioned
docs snapshot, which is not allowed outside a docs-freeze release-cut. Move this
content out of the `docs/v1.14.7` snapshot into the edge-only docs location, or
make the change only in a `[docs-freeze]` PR; use the `merged-step-card.mdx`
page content and its related assets/sections as the items to relocate.
In `@docs/v1.14.7/en/enterprise/features/merged-step-card.mdx`:
- Around line 1-83: This page was added under a frozen docs snapshot, which is
not allowed in a normal PR. Remove the new `merged-step-card.mdx` content from
the `docs/v1.14.7` snapshot path and place it in the appropriate non-frozen docs
source location instead; if this is a release-cut change, only keep it in the
frozen snapshot when the PR is explicitly marked `[docs-freeze]`. Use the
`merged-step-card.mdx` document itself as the target to relocate, preserving its
content outside `docs/v*`.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 8c6bfe26-58fc-4ab0-8269-438c87cc06b5
⛔ Files ignored due to path filters (3)
docs/images/enterprise/merged-step-card-canvas.pngis excluded by!**/*.pngdocs/images/enterprise/merged-step-card-editor.pngis excluded by!**/*.pngdocs/images/enterprise/merged-step-card-swap-agent.pngis excluded by!**/*.png
📒 Files selected for processing (9)
docs/docs.jsondocs/edge/ar/enterprise/features/merged-step-card.mdxdocs/edge/en/enterprise/features/merged-step-card.mdxdocs/edge/ko/enterprise/features/merged-step-card.mdxdocs/edge/pt-BR/enterprise/features/merged-step-card.mdxdocs/v1.14.7/ar/enterprise/features/merged-step-card.mdxdocs/v1.14.7/en/enterprise/features/merged-step-card.mdxdocs/v1.14.7/ko/enterprise/features/merged-step-card.mdxdocs/v1.14.7/pt-BR/enterprise/features/merged-step-card.mdx
…r (AGE-107) The page itself already landed on main via #6247. This rebases onto main and applies the two remaining changes: - Nest crew-studio + merged-step-card into a collapsible "Crew Studio" nav group (pencil icon), across edge and v1.14.7 in en, pt-BR, ko, ar. - Remove the temporary "Rolling out" Note banner (feature ships today). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
jessemiller
force-pushed
the
docs/age-107-merged-step-card
branch
from
d505b5d to
8be8e38
Compare
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx (1)
1-83: 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick winDo not modify frozen
docs/v*snapshots in this PR.This file is under a frozen release snapshot path and should not be changed unless this is a
[docs-freeze]release-cut PR. Please move this update todocs/edge/...only (or run it via a dedicated[docs-freeze]PR).As per coding guidelines, "Never modify files under
docs/v*/as they are frozen release snapshots, except in[docs-freeze]release-cut PRs".🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx` around lines 1 - 83, This update is targeting a frozen release snapshot under docs/v*, which should not be edited in this PR. Move the same Arabic merged-step-card content into the corresponding docs/edge documentation instead, or submit it in a dedicated [docs-freeze] release-cut PR if this change must land in the snapshot; keep the content aligned with the existing merged-step-card page structure when relocating it.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@docs/v1.14.7/ar/enterprise/features/merged-step-card.mdx`:
- Around line 1-83: This update is targeting a frozen release snapshot under
docs/v*, which should not be edited in this PR. Move the same Arabic
merged-step-card content into the corresponding docs/edge documentation instead,
or submit it in a dedicated [docs-freeze] release-cut PR if this change must
land in the snapshot; keep the content aligned with the existing
merged-step-card page structure when relocating it.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: b395c3b7-fd1a-4463-b874-1bbd772a27b2
📒 Files selected for processing (9)
docs/docs.jsondocs/edge/ar/enterprise/features/merged-step-card.mdxdocs/edge/en/enterprise/features/merged-step-card.mdxdocs/edge/ko/enterprise/features/merged-step-card.mdxdocs/edge/pt-BR/enterprise/features/merged-step-card.mdxdocs/v1.14.7/ar/enterprise/features/merged-step-card.mdxdocs/v1.14.7/en/enterprise/features/merged-step-card.mdxdocs/v1.14.7/ko/enterprise/features/merged-step-card.mdxdocs/v1.14.7/pt-BR/enterprise/features/merged-step-card.mdx
✅ Files skipped from review due to trivial changes (6)
- docs/edge/pt-BR/enterprise/features/merged-step-card.mdx
- docs/v1.14.7/pt-BR/enterprise/features/merged-step-card.mdx
- docs/v1.14.7/ko/enterprise/features/merged-step-card.mdx
- docs/edge/en/enterprise/features/merged-step-card.mdx
- docs/v1.14.7/en/enterprise/features/merged-step-card.mdx
- docs/edge/ko/enterprise/features/merged-step-card.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/docs.json
2 participants
What
Adds the One Card per Step Studio docs page (AGE-107) and nests it under Crew Studio in the AMP navigation.
Changes
enterprise/features/merged-step-cardexplaining the one-card-per-step Studio canvas change.crew-studio+merged-step-cardinto a collapsible Crew Studio group (with thepencilicon), so One Card per Step appears as a sub-page of Crew Studio.edgeandv1.14.7for all locales:en,pt-BR,ko,ar.Notes
.../enterprise/features/merged-step-card), so no links break.🤖 Generated with Claude Code
Note
Low Risk
Documentation and navigation JSON only; no runtime or API impact.
Overview
Reorganizes Enterprise Build navigation so Crew Studio and One Card per Step (
merged-step-card) sit under a nested Crew Studio group with thepencilicon, instead of two sibling top-level entries. The same structure is applied indocs/docs.jsonfor edge and v1.14.7 across en, pt-BR, ko, and ar (Arabic uses a localized group title).Removes the time-bound June 24th rollout
<Note>(and its cleanup comment) from everymerged-step-card.mdxcopy so the page reads as evergreen; page URLs are unchanged.Reviewed by Cursor Bugbot for commit 5d168f2. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit