Skip to content

ci: add GitHub Pages workflow for API docs#174

Closed
sachiniyer wants to merge 3 commits intomainfrom
ci/api-docs-pages-v3
Closed

ci: add GitHub Pages workflow for API docs#174
sachiniyer wants to merge 3 commits intomainfrom
ci/api-docs-pages-v3

Conversation

@sachiniyer
Copy link
Copy Markdown
Contributor

@sachiniyer sachiniyer commented Mar 11, 2026

Deploy the Scalar API reference to GitHub Pages on pushes to main
when openapi.json or docs/api/ changes. Copies both files into a
flat _site directory for the Pages artifact.

Requires enabling GitHub Pages with "GitHub Actions" as the source
in the repository settings.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com


Open with Devin

Copy link
Copy Markdown
Contributor Author

sachiniyer commented Mar 11, 2026

@sachiniyer sachiniyer marked this pull request as ready for review March 11, 2026 00:46
Base automatically changed from docs/scalar-api-reference-v3 to chore/sync-openapi-and-deps March 11, 2026 00:46
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

🐛 1 issue in files not directly in the diff

🐛 Workflow file is empty — no CI pipeline will run (.github/workflows/api-docs.yml:1)

The file .github/workflows/api-docs.yml is 0 bytes (empty), despite the commit message stating it should "Deploy the Scalar API reference to GitHub Pages on pushes to main when openapi.json or docs/api/ changes." An empty workflow file means GitHub Actions will not execute any workflow for API docs deployment. This appears to be a commit where the file was created but its contents were never written.

View 1 additional finding in Devin Review.

Open in Devin Review

sachiniyer and others added 3 commits March 10, 2026 17:50
Sync openapi.json with upstream and update quinn-proto from 0.11.13
to 0.11.14 to fix RUSTSEC-2026-0037 (DoS via invalid QUIC transport
parameters).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add a static HTML page that renders the OpenAPI spec using Scalar's
CDN-hosted API reference component. AI agent and client button are
disabled for a clean, focused documentation experience.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Deploy the Scalar API reference to GitHub Pages on pushes to main
when openapi.json or docs/api/ changes. Copies both files into a
flat _site directory for the Pages artifact.

Requires enabling GitHub Pages with "GitHub Actions" as the source
in the repository settings.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sachiniyer sachiniyer force-pushed the chore/sync-openapi-and-deps branch from 491531b to c8398f5 Compare March 11, 2026 00:50
@sachiniyer sachiniyer force-pushed the ci/api-docs-pages-v3 branch from 7ad3adc to e5f024a Compare March 11, 2026 00:50
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 new potential issues.

🐛 1 issue in files not directly in the diff

🐛 Workflow file is empty — no CI pipeline will run (.github/workflows/api-docs.yml:1)

The file .github/workflows/api-docs.yml is 0 bytes (empty), despite the commit message stating it should "Deploy the Scalar API reference to GitHub Pages on pushes to main when openapi.json or docs/api/ changes." An empty workflow file means GitHub Actions will not execute any workflow for API docs deployment. This appears to be a commit where the file was created but its contents were never written.

View 2 additional findings in Devin Review.

Open in Devin Review

Comment thread docs/api/index.html
<script src="https://cdn.jsdelivr.net/npm/@scalar/api-reference"></script>
<script>
Scalar.createApiReference("#app", {
url: "./openapi.json",
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.

🔴 Incorrect relative path to openapi.json — file is at repo root, not in docs/api/

The HTML page at docs/api/index.html references ./openapi.json (line 13), which resolves to docs/api/openapi.json. However, the actual openapi.json lives at the repository root (openapi.json), as confirmed by xtask/src/main.rs:5 (const OPENAPI_PATH: &str = "openapi.json"). Since there is no build step that copies the file into docs/api/ (the workflow file is empty), this reference will 404 at runtime and the Scalar API reference page will fail to load any spec.

Prompt for agents
The url in docs/api/index.html line 13 is set to "./openapi.json" but the openapi.json file lives at the repository root. Either:
1. Change the relative path to "../../openapi.json" to point to the repo root, OR
2. Add a build/deploy step in the (currently empty) .github/workflows/api-docs.yml that copies openapi.json into docs/api/ before deployment, OR
3. Use an absolute URL (e.g. the upstream https://api.detail.dev/public/v1/openapi.json).

Option 2 is likely intended since the workflow file was supposed to handle deployment.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Base automatically changed from chore/sync-openapi-and-deps to main March 11, 2026 00:53
@sachiniyer sachiniyer closed this Mar 11, 2026
@sachiniyer sachiniyer deleted the ci/api-docs-pages-v3 branch March 11, 2026 01:12
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