Skip to content

chore: archive an SDK and do a light clean-up round#2184

Merged
novusnota merged 5 commits into
mainfrom
novusnota/light-cleanup
May 25, 2026
Merged

chore: archive an SDK and do a light clean-up round#2184
novusnota merged 5 commits into
mainfrom
novusnota/light-cleanup

Conversation

@novusnota
Copy link
Copy Markdown
Collaborator

@novusnota novusnota commented May 25, 2026

npm run fmt, npm run spell, a hyphen replacement, an archival of ton-kotlin, and a cautionary tale on royalties.

Summary by CodeRabbit

  • Documentation

    • Enhanced style guide with expanded safety warnings, formatting requirements, and writing best practices
    • Clarified NFT reference implementation and wallet address verification documentation
    • Improved NFT comparison and system contract descriptions
    • Reorganized NFT-2.0 references with redirects to comparison page
  • Bug Fixes

    • Fixed typo in API endpoint documentation
    • Normalized typographic characters and hyphenation across documentation

Review Change Stack

@novusnota novusnota requested a review from a team as a code owner May 25, 2026 23:35
@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 25, 2026

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

Project Status Preview Updated (UTC)
mintlify-ton-docs 🟢 Ready View Preview May 25, 2026, 11:37 PM

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 25, 2026

Warning

Review limit reached

@novusnota, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 55 minutes and 3 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d7753056-b9e9-4ee2-a9fc-c2cd389a1b89

📥 Commits

Reviewing files that changed from the base of the PR and between 424a75e and 9f0e5be.

📒 Files selected for processing (1)
  • standard/tokens/nft/comparison.mdx
📝 Walkthrough

Walkthrough

This PR standardizes documentation across the TON docs repository: refines the contributing style guide with stricter safety and formatting guidelines, restructures NFT documentation by removing nft-2.0 and updating related redirects, clarifies NFT reference implementation and wallet authentication flows, and normalizes typography (ASCII hyphens) across 40+ files for encoding consistency.

Changes

Documentation Standards, Navigation, and Content Clarifications

Layer / File(s) Summary
Contributing standards and documentation guidelines
contribute/style-guide.mdx
Expanded guidance on answer-first structure, scannability, copy-paste-safe examples, explicit Caution/Warning callouts for fund/key/mainnet risks, root-absolute internal links, and Image component requirements with alt text and darkSrc handling.
Spellcheck configuration
.cspell.jsonc
Extended filename-specific overrides to ignore API-related terms (multisigs, Uninited) and docs.json references (subsecond).
NFT-2.0 deprecation in navigation and redirects
docs.json
Removed nft-2.0 from standard/tokens/nft navigation pages; updated redirects for /v3/documentation/dapps/assets/nft-2.0 and /standard/tokens/nft/nft-2.0 to point to comparison page.
NFT verification and comparison documentation
standard/tokens/nft/verify.mdx, standard/tokens/nft/comparison.mdx
Added off-chain API verification method subsection; expanded cNFT vs Default Collection capability table with eligibility, minting permissions, use cases, and trade-offs; clarified Single NFT bullet list.
NFT reference implementation clarification
standard/tokens/nft/nft-reference.mdx
Reflowed and clarified documentation for TEP-62/66 requirements, internal message headers (explicit non-bounceable explanation), royalty handling, collection and item get-method behavior, and metadata composition.
NFT overview and fundamentals documentation
standard/tokens/nft/overview.mdx, standard/tokens/nft/how-it-works.mdx
Standardized "Non-Fungible" terminology and best-practice punctuation across NFT introduction and fundamentals sections.
Wallet authentication and configuration clarification
standard/wallets/how-it-works.mdx, standard/wallets/interact.mdx, standard/wallets/mnemonics.mdx, standard/wallets/preprocessed-v2/specification.mdx
Enhanced Ed25519 signature verification explanation, clearer subwallet_id/wallet_id descriptions, improved mnemonic-format comments, and expanded wrap-around safety guidance for seq_no replay protection.
Typography and encoding standardization across documentation
contract-dev/blueprint/*, ecosystem/**/*, foundations/**/*, standard/**/*, tolk/**/*, tvm/**/*
Systematic normalization of non-ASCII hyphen characters to standard ASCII hyphens (e.g., on‑chainon-chain, TL‑BTL-B) and apostrophe/quote standardization across 40+ files for encoding consistency.
Documentation layout and spacing refinements
ecosystem/tma/analytics/*, ecosystem/tma/telegram-ui/*, ecosystem/explorers/overview.mdx, ecosystem/status.mdx
Minor spacing adjustments between sections and table reformatting for improved visual clarity (blank-line separation in install guides, markdown table alignment).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • ton-org/docs#2104: Reorders NFT navigation entries in docs.json; this PR removes nft-2.0 from the same navigation section and updates related redirects.

Suggested reviewers

  • it09
  • kay-is
  • delovoyhomie

Poem

🐰 A rabbit hops through docs with care,
Fixing hyphens everywhere!
NFT-2.0 takes its final bow,
ASCII hyphens shining now.

🚥 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 accurately reflects the main changes: archiving SDK content (ton-kotlin removal from sdks.mdx and nft-2.0 deprecation) and performing documentation cleanup (typography normalization, rewording, formatting fixes across 30+ files).
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch novusnota/light-cleanup

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 and usage tips.

github-actions[bot]

This comment was marked as off-topic.

@novusnota novusnota merged commit 2be7f48 into main May 25, 2026
11 checks passed
@novusnota novusnota deleted the novusnota/light-cleanup branch May 25, 2026 23:40
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