Skip to content

Reorder language guides and reposition category in docs nav#962

Merged
maryelizbeth merged 1 commit into
mainfrom
guide-order
Jul 23, 2026
Merged

Reorder language guides and reposition category in docs nav#962
maryelizbeth merged 1 commit into
mainfrom
guide-order

Conversation

@maryelizbeth

Copy link
Copy Markdown
Contributor

Closes MIR-1442.

What

Language guides were in effectively random order, and the ~38-item category sat high in the left-nav dominating the top of the sidebar.

  • Ordering (sidebar + guides/index.md table now match): auto-detected stacks first (Python, JavaScript, Go, Ruby, Rust), then Dockerfile languages by rough relevance, with an esoteric tail (Klong, COBOL, Bash, Static).
  • Nav placement: moved Language Guides and Recipes down to sit just above Reference. New order: intro → getting-started → Deploy → Data & Storage → Networking & Security → Run & Scale → Miren Cloud → Language Guides → Recipes → Reference → Resources.
  • No longer a landing page: the category no longer uses a link: {doc}; the index content now sits under an Overview item as the first entry (mirroring the Miren Cloud pattern).

The page heading stays "Language Guides" for now.

Testing

bun run lint passes.

Order guides auto-detected-first, then by relevance, with an esoteric
tail. Move Language Guides and Recipes down to sit just above Reference,
and turn Language Guides into a plain category with an Overview item
instead of a landing-page link.

MIR-1442
@maryelizbeth
maryelizbeth requested a review from a team as a code owner July 23, 2026 20:16
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4125cc09-afbd-47de-8181-dee2c2066eb4

📥 Commits

Reviewing files that changed from the base of the PR and between 8a3afae and f78f5a5.

📒 Files selected for processing (2)
  • docs/docs/guides/index.md
  • docs/sidebars.ts

📝 Walkthrough

Walkthrough

The language guide table now reorders several Dockerfile.miren entries and adds JRuby and TruffleRuby. The documentation sidebar relocates the Language Guides and Recipes categories, adds an explicit Overview item, and reorganizes the guide document order into grouped sections.


Comment @coderabbitai help to get the list of available commands.

@miren-code-agent miren-code-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍪 biscuit: ✅ ready to merge — auto-review, non-blocking

This is a clean documentation-only reorganization across two files. I checked it carefully:

What changed:

  1. sidebars.ts — "Language Guides" moves from position 3 (right after Deploy) to near the bottom, sitting between "Miren Cloud" and "Recipes". The category's link: property (which made guides/index the category header) is replaced with an explicit 'Overview' doc item as the first child — a reasonable and functionally equivalent swap; the index page stays reachable.
  2. docs/guides/index.md — The "Pick your language" table is reordered to match the new sidebar order: auto-detected stacks first (Python, JS, Go, Ruby, Rust), then Dockerfile-based languages roughly by relevance (Java, PHP, .NET, C++, C, Deno, Elixir, … down to the niche tail).

Consistency check: I enumerated both lists entry-by-entry. Every guide present in the markdown table is present in the sidebar and in the same relative order, with nothing added or dropped. The sidebar's 'Overview' item doesn't appear in the table (it's not a language, it's the index page itself), which is correct.

One minor thing to be aware of: removing the link: from the category means the category label in the sidebar is no longer itself a clickable link to the Overview page — readers have to expand the category and click "Overview". In Docusaurus this is expected behavior when link: is absent, so it's not a bug, just a UX trade-off. Given there's now an explicit "Overview" item as the first child, this seems intentional and fine.

No correctness issues, no missing entries, no broken structure. This is ready to merge.


🍪 full review note · comment /biscuit review to run biscuit again.

@phinze phinze left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📄 🪄

@maryelizbeth
maryelizbeth merged commit 8360187 into main Jul 23, 2026
23 checks passed
@maryelizbeth
maryelizbeth deleted the guide-order branch July 23, 2026 21:38
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.

2 participants