Skip to content

chore(docs): stop tracking generated OpenAPI doc output - #287

Merged
joalves merged 2 commits into
masterfrom
chore/remove-generated-api-docs
Aug 18, 2026
Merged

chore(docs): stop tracking generated OpenAPI doc output#287
joalves merged 2 commits into
masterfrom
chore/remove-generated-api-docs

Conversation

@joalves

@joalves joalves commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the committed output of docusaurus-plugin-openapi-docs (*.api.mdx, *.info.mdx, sidebar.js) under docs/APIs-and-SDKs/SDK-API and docs/APIs-and-SDKs/Web-Console-API
  • Fixes .gitignore so this output (and its JSON sidecars / sidebar.ts) stays untracked going forward — the previous patterns pointed at the wrong paths and didn't cover the plugin's newer per-endpoint JSON files
  • Split out from chore(deps): upgrade Docusaurus 2.4.3 to 3.10.x (FT-2118) #284 so that PR's diff is limited to actual source changes instead of ~300 regenerated files

Test plan

  • yarn gen:api regenerates the docs locally and git status stays clean (confirms the ignore patterns match)
  • yarn build succeeds

Summary by CodeRabbit

  • Documentation

    • Removed outdated SDK and Web Console API reference pages, including endpoint descriptions, request and response details, permissions, schemas, and examples.
    • Removed associated API documentation metadata and navigation entries.
    • The documentation area now contains fewer legacy API references.
  • Chores

    • Updated repository ignore rules to target generated API documentation files while preserving category metadata and Web Console examples.

The docusaurus-plugin-openapi-docs output (*.api.mdx, *.info.mdx,
sidebar.js, and their JSON sidecars) is regenerated by `yarn gen:api`
and shouldn't be committed. Removing it now, ahead of the Docusaurus
v3 upgrade, keeps that PR's diff to actual source changes instead of
hundreds of regenerated files.
@coderabbitai

coderabbitai Bot commented Aug 17, 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: 2be7eb4f-f293-4462-b63b-b6e45e003af8

📥 Commits

Reviewing files that changed from the base of the PR and between 18daa10 and 1f91372.

📒 Files selected for processing (1)
  • .gitignore
🚧 Files skipped from review as they are similar to previous changes (1)
  • .gitignore

Included review availability: 4 reviews are currently available. Based on recent review activity, included reviews refill at 5 per hour.


Walkthrough

The change removes generated SDK API and Web Console API documentation pages, including endpoint metadata, schemas, examples, and response details. It also removes the related sidebar exports. The .gitignore entries now target generated documentation directories while preserving _category_.json files and the Web Console Examples directory.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 1f913

This change stops tracking generated API documentation while preserving regeneration through updated ignore rules; no actionable merge-blocking risk remains after normal checks.

Poem

A rabbit hopped through docs today,
And watched generated pages fade away.
The sidebars rest, the rules are bright,
Category files remain in sight.
“A tidy burrow!” the rabbit says.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing generated OpenAPI documentation from tracking.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-generated-api-docs

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

@netlify

netlify Bot commented Aug 17, 2026

Copy link
Copy Markdown

Deploy Preview for absmartly-docs ready!

Name Link
🔨 Latest commit 1f91372
🔍 Latest deploy log https://app.netlify.com/projects/absmartly-docs/deploys/6a843193221e93000821535a
😎 Deploy Preview https://deploy-preview-287--absmartly-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread .gitignore Outdated
Address review feedback (PR #287): explicitly listing every generated
filename pattern is fragile — it silently misses whatever new sidecar
file a future plugin bump emits, exactly like the pre-upgrade .gitignore
missed the *.ParamsDetails.json/etc files this branch had to clean up.
Ignore everything in the two output directories and carve back out only
the hand-authored files (_category_.json, the Examples/ page).
@joalves
joalves merged commit 3041b8d into master Aug 18, 2026
6 checks passed
@joalves
joalves deleted the chore/remove-generated-api-docs branch August 18, 2026 11:23
joalves added a commit that referenced this pull request Aug 18, 2026
…l SVGs

Rebased onto master now that #282/#287/#288 landed. Swaps the inline SVG
diagrams from #282 for editorial versions built via the diagram-design
skill: real infra icons (phone, database, bucket, filter, search, cloud,
chart) instead of plain labeled boxes, and the Hybrid diagram's
ABsmartly Cloud node correctly reflects that it still holds managed-
source data rather than reusing Fully mode's "no user-level data
stored" claim. Prose and topology are otherwise unchanged from master.
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