Skip to content

docs: prefer OAuth over Auth0 in user-facing copy#124

Merged
k-taro56 merged 2 commits intomainfrom
eng-672
May 7, 2026
Merged

docs: prefer OAuth over Auth0 in user-facing copy#124
k-taro56 merged 2 commits intomainfrom
eng-672

Conversation

@soleil-colza
Copy link
Copy Markdown
Contributor

Summary

Sweep through the docs prose so user-facing copy says OAuth (the verb the
user touches) instead of Auth0 (the underlying provider). Two commits:

  • docs(roadmap): rename the Auth0 token auto-refresh card to OAuth
    token auto-refresh
    in docs/roadmap.mdx + docs/ja/roadmap.mdx. Anchor
    renamed to #oauth-token-auto-refresh and the two inbound links in
    docs/cli/auth.mdx + docs/ja/cli/auth.mdx updated to match.
  • docs: replace Auth0 with OAuth in cli/dev.mdx + cli/auth.mdx
    prose (and JA mirrors):
    • cli/dev.mdx: "no Auth0 advertised in /v1/auth/cli/config" →
      "no OAuth advertised in …"
    • cli/auth.mdx: "Enable the offline_access scope on the OAuth (Auth0)
      application" → "OAuth application"

Intentionally not changed

These are runtime-contract values, exported type names, or verbatim strings
the CLI itself emits — substituting "OAuth" would either break behaviour or
desync from the source:

  • The literal mode: "auth0" value in ~/.arkor/credentials.json (and
    every doc reference to it).
  • The Auth0Credentials SDK type name and the parenthetical that explains
    it is "named after the underlying Auth0-based provider".
  • The verbatim Auth0 did not return a refresh token. Make sure the Application has 'offline_access' scope enabled. error string emitted by
    packages/arkor/src/core/auth0.ts.

Test plan

  • pnpm --filter docs build (or Mintlify preview) renders the roadmap
    page with the renamed card, and the cli/auth "Token expiry" link
    lands on the new #oauth-token-auto-refresh anchor in both locales.
  • git grep -n "Auth0" docs/ shows only the intentionally-kept
    occurrences listed above.

…g copy

The roadmap card and the cli/auth roadmap link both said "Auth0";
user-facing surfaces use OAuth as the verb (Auth0 is the provider, not
the noun the user touches). Anchor renamed to `#oauth-token-auto-refresh`
and the two inbound links in cli/auth.mdx updated to match.
Sweep through the docs prose so user-facing copy says OAuth (the verb
the user touches) instead of Auth0 (the underlying provider). Limited
to non-literal references; left untouched are:

- the literal `mode: "auth0"` value in `~/.arkor/credentials.json`
- the `Auth0Credentials` SDK type name and its provenance note
- the verbatim `Auth0 did not return a refresh token...` error string
  emitted by the CLI source

Updates:
- cli/dev.mdx + JA: "no Auth0 advertised in `/v1/auth/cli/config`" ->
  "no OAuth advertised in `/v1/auth/cli/config`"
- cli/auth.mdx + JA Fix column: "OAuth (Auth0) application" ->
  "OAuth application"
@soleil-colza soleil-colza self-assigned this May 7, 2026
Copilot AI review requested due to automatic review settings May 7, 2026 11:33
@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 7, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
arkor-92aeef0e 🟢 Ready View Preview May 7, 2026, 11:36 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates user-facing documentation to consistently describe the sign-in experience in terms of OAuth (what users interact with) rather than Auth0 (the underlying provider), while preserving intentional runtime-contract strings and type names.

Changes:

  • Renames the roadmap card and anchor from “Auth0 token auto-refresh” to “OAuth token auto-refresh”, and updates inbound links accordingly (EN/JA).
  • Replaces remaining user-facing “Auth0 advertised …” phrasing with “OAuth advertised …” in CLI docs troubleshooting prose (EN/JA).
  • Simplifies the “offline_access” remediation copy to refer to the “OAuth application” (EN/JA), while leaving the verbatim CLI error string unchanged.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/roadmap.mdx Renames OAuth auto-refresh roadmap card + updates anchor id.
docs/ja/roadmap.mdx Japanese mirror of the roadmap card title + anchor id change.
docs/cli/dev.mdx Updates anon-only deployment troubleshooting copy to say “OAuth” instead of “Auth0”.
docs/ja/cli/dev.mdx Japanese mirror of the same troubleshooting copy adjustment.
docs/cli/auth.mdx Updates roadmap anchor link + refines offline_access remediation wording.
docs/ja/cli/auth.mdx Japanese mirror of anchor link update + remediation wording tweak.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@soleil-colza soleil-colza requested a review from k-taro56 May 7, 2026 15:45
Copy link
Copy Markdown
Contributor

@k-taro56 k-taro56 left a comment

Choose a reason for hiding this comment

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

エンジニアを褒めるネコ_LGTM

@k-taro56 k-taro56 enabled auto-merge May 7, 2026 15:46
@k-taro56 k-taro56 disabled auto-merge May 7, 2026 15:48
@k-taro56 k-taro56 merged commit 2abfc09 into main May 7, 2026
72 checks passed
@k-taro56 k-taro56 deleted the eng-672 branch May 7, 2026 15:48
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.

3 participants