Skip to content

docs(providers): update TokenLab docs links - #6474

Merged
lidge-jun merged 1 commit into
lidge-jun:devfrom
hedging8563:codex/tokenlab-docs-canonical-links
Oct 3, 2026
Merged

lidge-jun merged 1 commit into
lidge-jun:devfrom
hedging8563:codex/tokenlab-docs-canonical-links

Conversation

@hedging8563

@hedging8563 hedging8563 commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

TokenLab documentation moved to the self-hosted https://tokenlab.sh/docs site. The TokenLab provider guide still sends readers through the former docs subdomain. Update its six guide, quickstart and API reference links to the current direct addresses, preserving the Korean guide link and each document destination.

Verification

  • Reviewed the one-file diff: documentation URLs only; provider configuration, API endpoint, model presets and referral parameters are unchanged.
  • Current public TokenLab documentation targets are reachable.
  • Remote isolated validation on the exact head 4233cb5 passed with the project-declared Bun 1.4.0: cd docs-site, bun install --frozen-lockfile, bun run build. Built 561 pages and checked 77,818 internal links. The task container was removed after completion.
  • Runtime full-suite exception: this PR changes only six URLs in one Markdown guide, with no CLI, GUI, provider implementation or configuration change; the docs build covers the affected consumer.
  • The PR head is based on the current dev tip. No Codex/CodeRabbit findings have been raised; CodeRabbit skipped automatic review while this was a draft.
  • Upstream Cross-platform CI and React Doctor await maintainer workflow approval; their results are not claimed as passed.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • Required local validation passed; commands, results, and any full-suite exception are documented.

  • I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

Summary by CodeRabbit

  • Documentation
    • Updated TokenLab documentation links in the provider guide to use the current site address.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: lidge-jun/opencodex/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 689f85ee-19a9-479f-8239-35ddf4675162
📥 Commits

Reviewing files that changed from the base of the PR and between b4616be and 4233cb5.

📒 Files selected for processing (1)
  • docs-site/src/content/docs/guides/providers.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The TokenLab provider guide now links to tokenlab.sh for integration guides, API references, and model discovery. The documented model-format routing and modelAdapters behavior remain unchanged.

Changes

TokenLab documentation links

Layer / File(s) Summary
Update TokenLab documentation links
docs-site/src/content/docs/guides/providers.md:605-607, docs-site/src/content/docs/guides/providers.md:616-625
The integration guides, Chat Completions and model-discovery references, Responses and Messages references, and model catalog now use the tokenlab.sh domain. Routing guidance remains unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 4233c

Readers can reach the intended TokenLab guides and references through the updated links. No merge-blocking issue was identified.

Architecture Summary

Architecture risk: 🔵 Low · up to 4233c

The change affects 1 system.

Changed systems: docs-site

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs-site (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in docs-site/src/content/docs/guides/providers.md: Changed the TokenLab integration guide, Korean guide, Chat Completions, and model-discovery links to the tokenlab.sh domain; the documented routing behavior is unchanged.
  • observed — Modified behavior in docs-site/src/content/docs/guides/providers.md: Updated TokenLab Responses, Messages, and model-catalog documentation links from docs.tokenlab.sh to tokenlab.sh. The model-format table and provider routing guidance remain unchanged.
🚥 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 primary change: updating TokenLab documentation links in the provider guide.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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 a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 2, 2026
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

✅ READY

  • all PR quality gates passed; the review readiness checklist is complete.

Review readiness checklist

  • ✅ Required local validation passed; commands, results, and any full-suite exception are documented.
  • ✅ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
  • ✅ I resolved all correct Codex and CodeRabbit findings.
  • ✅ My PR is ready for review.

✅ 4/4 boxes ticked.

This pull request is already Ready for Review.
The review-ready label marks this PR as ready; review automation runs independently.
Maintainers: @lidge-jun @Ingwannu

@hedging8563
hedging8563 marked this pull request as ready for review October 2, 2026 23:30
@Ingwannu

Ingwannu commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Link-only review at 4233cb5: I read the entire six-link diff and independently fetched all six replacement URLs without credentials. Each returns the expected TokenLab documentation title (English/Korean OpenCodex, Quickstart, Create Response, Create Message, List Models), rather than a generic homepage. This is evidence for the documentation relocation only, not renewed endorsement of provider access, pricing, or every advertised model. No inference, account action, build or security scan was performed. Required current-head CI is still a separate readiness gate; I have not marked an unrun docs build as passing.

@lidge-jun lidge-jun left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Reviewed docs-site/src/content/docs/guides/providers.md at 4233cb5 (all changed lines, 605-609, 616-617 and 625).

No actionable findings. The diff contains exactly six documentation URL replacements, with surrounding Markdown, provider configuration, credential destination, model presets and referral parameters unchanged. All six new HTTPS targets returned HTTP 200 and the expected English/Korean integration, quickstart or API-reference content on 2026-10-03.

Verification: Cross-platform CI run 37077313656, attempt 2, pull_request event, head 4233cb5 succeeded, including docs site build (bun install --frozen-lockfile; bun run build), privacy gate and aggregate ci. React Doctor run 37077946385, attempt 2, on the same head succeeded. Enforce-target and hygiene passed; there are no unresolved review threads. Runtime/platform suites were correctly skipped by the docs-only scope filter and are not claimed as passing.

blocking_issues: none.

Approve for squash merge into dev.

@lidge-jun
lidge-jun merged commit 2e3acab into lidge-jun:dev Oct 3, 2026
35 of 36 checks passed
robin-bially pushed a commit to robin-bially/opencodex that referenced this pull request Oct 3, 2026
…#6474)

Replace six relocated guide and API-reference URLs with their direct destinations.
Provider endpoints, routing, and configuration remain unchanged.

Carries lidge-jun#6474 by @hedging8563.

Co-authored-by: hedging8563 <45883076+hedging8563@users.noreply.github.com>
@lidge-jun lidge-jun mentioned this pull request Oct 4, 2026
3 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation review-ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants