Skip to content

release: satisfy validate-docs for v1.2.7#596

Merged
arul28 merged 2 commits into
mainfrom
ade/ok-please-pull-from-main-remote-e8c0281f
Jun 17, 2026
Merged

release: satisfy validate-docs for v1.2.7#596
arul28 merged 2 commits into
mainfrom
ade/ok-please-pull-from-main-remote-e8c0281f

Conversation

@arul28

@arul28 arul28 commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Adds the brand PNGs (gitignored), v1.2.7 CHANGELOG.md entry + link ref, and changelog index card. Unblocks the v1.2.7 release verify job. Docs-only. 🤖 Generated with Claude Code

Greptile Summary

  • Adds v1.2.7 release notes and changelog navigation updates.
  • Reworks the Mintlify docs structure with updated pages for onboarding, chat, configuration, integrations, tools, and references.
  • Updates docs branding assets, favicon, and logo configuration.

Confidence Score: 4/5

The release documentation update is mostly safe, but a couple of existing product links now point to removed documentation routes.

The changed docs structure and route references were checked closely enough to identify the remaining broken-link risk without indications of broader runtime impact.

docs.json and the hardcoded docs-link callers in apps/desktop/src/renderer/onboarding/docsLinks.ts and apps/web/src/components/editorial/IndexPage.tsx need attention.

T-Rex T-Rex Logs

What T-Rex did

  • I ran the generated Node docs-link validation script against the checked-out repository files.
  • The script reported a hardcoded link to /docs/lanes/packs with no matching route, and a hardcoded link to /docs/cto/linear with no matching route.
  • The script confirmed the Linear docs route exists at /docs/integrations/linear, replacing the stale /docs/cto/linear link.
  • I compared the before/after docs validation logs; the after run shows documentation validation passed for 165 files with exit code 0.
  • These results support that the PR unblocks the v1.2.7 release docs validation checks for Mintlify branding assets and changelog material.
  • I captured before and after changelog UI states; the after-state renders the v1.2.7 page and loads the committed logo asset, and the server startup is visible in the logs.

View all artifacts

T-Rex Ran code and verified through T-Rex

Comments Outside Diff (1)

  1. docs.json, line 57-61 (link)

    P2 Deleted docs links remain

    This docs restructure removes or moves pages that are still referenced by hardcoded product and web links. apps/desktop/src/renderer/onboarding/docsLinks.ts still exposes docs.lanesPacks as /docs/lanes/packs after lanes/packs.mdx was removed, and apps/web/src/components/editorial/IndexPage.tsx still links Linear sync to /cto/linear after the Linear docs moved to integrations/linear.mdx. Those flows now send users to deleted pages. Please either restore or alias these old routes, or update the product links to the replacement pages.

    Artifacts

    Repro: generated docs link validation script

    • Contains supporting evidence from the run (text/javascript; charset=utf-8).

    Repro: failing docs link validation output

    • Keeps the command output available without making the summary code-heavy.

    View artifacts

    T-Rex Ran code and verified through T-Rex

    Prompt To Fix With AI
    This is a comment left during a code review.
    Path: docs.json
    Line: 57-61
    
    Comment:
    **Deleted docs links remain**
    
    This docs restructure removes or moves pages that are still referenced by hardcoded product and web links. `apps/desktop/src/renderer/onboarding/docsLinks.ts` still exposes `docs.lanesPacks` as `/docs/lanes/packs` after `lanes/packs.mdx` was removed, and `apps/web/src/components/editorial/IndexPage.tsx` still links `Linear sync` to `/cto/linear` after the Linear docs moved to `integrations/linear.mdx`. Those flows now send users to deleted pages. Please either restore or alias these old routes, or update the product links to the replacement pages.
    
    How can I resolve this? If you propose a fix, please make it concise.

    Fix in Claude Code

Fix All in Claude Code

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
docs.json:57-61
**Deleted docs links remain**

This docs restructure removes or moves pages that are still referenced by hardcoded product and web links. `apps/desktop/src/renderer/onboarding/docsLinks.ts` still exposes `docs.lanesPacks` as `/docs/lanes/packs` after `lanes/packs.mdx` was removed, and `apps/web/src/components/editorial/IndexPage.tsx` still links `Linear sync` to `/cto/linear` after the Linear docs moved to `integrations/linear.mdx`. Those flows now send users to deleted pages. Please either restore or alias these old routes, or update the product links to the replacement pages.

Reviews (1): Last reviewed commit: "release: satisfy validate-docs for v1.2...." | Re-trigger Greptile

arul28 and others added 2 commits June 16, 2026 22:21
Rebuild the public docs around ADE's real feature set (grounded in
docs/features/**): correct lane model (primary lane + worktree/child/
import/attach), add Review, iOS Simulator, Automations, GitHub, and
troubleshooting pages, consolidate Integrations (GitHub + Linear) and
AI Connections, swap the stale green logo for the real purple brand,
and wire the full /media library into the pages. Adds the v1.2.7
changelog. Docs-only — no app code touched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Force-add the brand favicon.png + logo/ade-wordmark.png (the repo
gitignores *.png), add the v1.2.7 CHANGELOG.md section + link
reference, and point the changelog index card at v1.2.7. Unblocks the
release verify job (ci-pass requires green validate-docs on the tag).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ade Ignored Ignored Preview Jun 17, 2026 2:30am

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@arul28, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 51 minutes and 17 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 92b1b101-4b77-40ec-af80-632551b10862

📥 Commits

Reviewing files that changed from the base of the PR and between 3e5216f and 65e7872.

⛔ Files ignored due to path filters (15)
  • CHANGELOG.md is excluded by !*.md
  • architecture.mdx is excluded by !*.mdx
  • changelog/index.mdx is excluded by !changelog/**
  • changelog/v1.2.7.mdx is excluded by !changelog/**
  • docs.json is excluded by !docs.json
  • favicon.png is excluded by !**/*.png
  • favicon.svg is excluded by !**/*.svg, !favicon.svg
  • index.mdx is excluded by !*.mdx
  • introduction.mdx is excluded by !*.mdx
  • key-concepts.mdx is excluded by !*.mdx
  • logo/ade-wordmark.png is excluded by !**/*.png, !logo/**
  • logo/dark.svg is excluded by !**/*.svg, !logo/**
  • logo/light.svg is excluded by !**/*.svg, !logo/**
  • quickstart.mdx is excluded by !*.mdx
  • welcome.mdx is excluded by !*.mdx
📒 Files selected for processing (52)
  • .mintignore
  • ai-tools/claude-code.mdx
  • ai-tools/cursor.mdx
  • ai-tools/windsurf.mdx
  • chat/capabilities.mdx
  • chat/context.mdx
  • chat/grid.mdx
  • chat/overview.mdx
  • computer-use/config.mdx
  • computer-use/overview.mdx
  • computer-use/proofs.mdx
  • computer-use/setup.mdx
  • configuration/ai-providers.mdx
  • configuration/overview.mdx
  • configuration/permissions.mdx
  • configuration/settings.mdx
  • cto/linear.mdx
  • cto/overview.mdx
  • cto/setup.mdx
  • cto/workers.mdx
  • getting-started/connect-provider.mdx
  • getting-started/first-agent.mdx
  • getting-started/first-lane.mdx
  • getting-started/install.mdx
  • getting-started/open-project.mdx
  • getting-started/project-setup.mdx
  • guides/cto-linear-workflow.mdx
  • guides/multi-agent-setup.mdx
  • guides/stacked-prs.mdx
  • integrations/github.mdx
  • integrations/linear.mdx
  • lanes/creating.mdx
  • lanes/environment.mdx
  • lanes/overview.mdx
  • lanes/packs.mdx
  • lanes/stacks.mdx
  • reference/cli.mdx
  • reference/troubleshooting.mdx
  • tools/ade-code.mdx
  • tools/automations.mdx
  • tools/browser.mdx
  • tools/conflicts.mdx
  • tools/files-editor.mdx
  • tools/history.mdx
  • tools/ios-companion.mdx
  • tools/ios-simulator.mdx
  • tools/project-home.mdx
  • tools/pull-requests.mdx
  • tools/remote-machines.mdx
  • tools/review.mdx
  • tools/terminals.mdx
  • tools/workspace-graph.mdx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ade/ok-please-pull-from-main-remote-e8c0281f

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.

@arul28
arul28 merged commit 413a30a into main Jun 17, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant