Skip to content

[Styler] Weekly style audit (2026-04-22)#1252

Merged
SahilAujla merged 1 commit intomainfrom
docs/styler/weekly-style-audit-2026-04-22
Apr 22, 2026
Merged

[Styler] Weekly style audit (2026-04-22)#1252
SahilAujla merged 1 commit intomainfrom
docs/styler/weekly-style-audit-2026-04-22

Conversation

@JackReacher0807
Copy link
Copy Markdown
Collaborator

Weekly style audit of content files changed in PRs merged during the last 7 days, against STYLE_RULES.md.

Fixes

Rebrand leftovers: Smart Wallets (product) → Wallet APIs

PR #1223 renamed the product surface from Smart Wallets to Wallet APIs on 2026-04-16. PR #1233 (merged later the same week) introduced two new pages that still referenced the old product name.

  • content/tutorials/build-with-ai/alchemy-cli.mdx
    • Line 8 (intro prose): nodes, Data APIs, Smart Wallets, webhooksnodes, Data APIs, Wallet APIs, webhooks
    • Line 163 (section heading): ### Smart Wallets (Bundler and Gas Manager)### Wallet APIs (Bundler and Gas Manager)
  • content/tutorials/build-with-ai/overview.mdx
    • Line 33: Data APIs, Smart Wallets, Notify webhooksData APIs, Wallet APIs, Notify webhooks

Note: lowercase smart wallet (the smart-contract-account concept) is preserved — only the product name was rebranded.

Terminology: on-chainonchain (prose)

Per the preferred-forms table in STYLE_RULES.md.

  • content/wallets/pages/index.mdx line 62 — land on-chainland onchain
  • content/wallets/pages/resources/types.mdx line 204 — sent to the bundler for on-chain execution...onchain execution
  • content/wallets/pages/resources/types.mdx line 356 — use the smart wallet on-chain...smart wallet onchain

Both files were touched by recent PRs (#1251, #1223, #1243), so they fall within this week's audit scope.

Not fixed (intentional)

  • content/wallets/pages/third-party/smart-contracts.mdx line 309 — "could not get on-chain owner" lives inside a code block error string; Styler does not modify code without verification.
  • content/wallets/shared/react-native/signer-setup.mdx line 82 — alt="how to copy the api key" is an image alt attribute; cosmetic only.
  • content/tutorials/getting-started/developer-best-practices/best-practices-when-using-alchemy.mdx has pre-existing API Key casing and duplicated ## 8. headings, but PR [codex] add websocket billing callouts #1218 only added callouts — the violations were not introduced this week.
  • content/wallets/pages/core/quickstart.mdx line 34 — copy the **API Key** reads as a UI label reference; left as-is.

Audit scope

gh pr list --state merged --search "merged:>=2026-04-15" --limit 50 returned 24 PRs; 147 unique .mdx/.md content files changed. All were scanned for terminology, casing, and tone violations.

- Rebrand leftovers: Smart Wallets → Wallet APIs (product name) in
  alchemy-cli.mdx and overview.mdx under content/tutorials/build-with-ai/
- Terminology: on-chain → onchain in wallets index.mdx and resources/types.mdx
@JackReacher0807 JackReacher0807 requested a review from a team as a code owner April 22, 2026 17:17
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

🔗 Preview Mode

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Apr 22, 2026, 5:18 PM

@github-actions
Copy link
Copy Markdown

🔍 Link Check

Status: ❌ Failed

Summary

Status Count
🔍 Total 7917
✅ Successful 6830
⏳ Timeouts 0
🔀 Redirected 409
👻 Excluded 676
❓ Unknown 0
🚫 Errors 1
⛔ Unsupported 1
Broken links (1) — click to expand

Errors per input

Errors in ./content/api-reference/arbitrum-nova/arbitrum-nova-deprecation-notice.mdx

View workflow run

@SahilAujla SahilAujla merged commit f0db4cd into main Apr 22, 2026
12 of 13 checks passed
@SahilAujla SahilAujla deleted the docs/styler/weekly-style-audit-2026-04-22 branch April 22, 2026 20:35
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