docs(sdk): Reorganize getting-started section and add new categories#16421
docs(sdk): Reorganize getting-started section and add new categories#16421stephanie-anderson merged 2 commits intomasterfrom
Conversation
Replace manual section listing with PageGrid, update philosophy description, and add placeholder sections for playbooks, standards, skills, and templates. Co-Authored-By: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| ## Principles | ||
|
|
||
| Working [principles](principles/) for how we collaborate, review, and ship. They translate philosophy into daily decisions: who owns what, when to automate vs. judge, and why small PRs beat big ones. | ||
| <PageGrid /> |
There was a problem hiding this comment.
Duplicate sidebar_order causes non-deterministic PageGrid ordering
Medium Severity
Switching from manually ordered links to <PageGrid /> exposes a sidebar_order collision: both philosophy and principles have sidebar_order: 2. Previously the index page explicitly listed Philosophy before Principles, so the collision was harmless. Now <PageGrid /> auto-orders children by sidebar_order, making the relative order of these two sections non-deterministic. The new sections use carefully sequenced values (3, 4, 5, 6), but this collision at 2 was not addressed.
Additional Locations (1)
| sidebar_order: 1 | ||
| --- | ||
|
|
||
| ## Philosophy |
There was a problem hiding this comment.
I do support this change. It was weird anyways that this was not linked to the Philosophy (and likewise for Principles) page 👍
| @@ -0,0 +1,7 @@ | |||
| --- | |||
| title: Skills | |||
There was a problem hiding this comment.
I wander if we wanna limit this to skills or add a more general (AI) tools section?
## DESCRIBE YOUR PR Follow-up to #16421. Applies changes that landed after the squash merge: - Rename `skills/` section to `ai-tools/` - Update philosophy sidebar_order from 2 to 1 - Update templates sidebar_order from 6 to 5 ## IS YOUR CHANGE URGENT? Help us prioritize incoming PRs by letting us know when the change needs to go live. - [ ] Urgent deadline (GA date, etc.): - [ ] Other deadline: - [x] None: Not urgent, can wait up to 1 week+ ## SLA - Teamwork makes the dream work, so please add a reviewer to your PRs. - Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it. Thanks in advance for your help! ## PRE-MERGE CHECKLIST *Make sure you've checked the following before merging your changes:* - [ ] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
2 participants
DESCRIBE YOUR PR
Reorganize the SDK getting-started section:
<PageGrid />IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
Co-Authored-By: Claude noreply@anthropic.com