Skip to content

Restore BRC-190 as the number for Access Gates for Metanet Rooms - #232

Open
vincemedia wants to merge 1 commit into
bsv-blockchain:masterfrom
vincemedia:brc-190-restore-number
Open

Restore BRC-190 as the number for Access Gates for Metanet Rooms#232
vincemedia wants to merge 1 commit into
bsv-blockchain:masterfrom
vincemedia:brc-190-restore-number

Conversation

@vincemedia

Copy link
Copy Markdown
Contributor

What this does

The Access Gates document was submitted as BRC-190 in #191 and merged as BRC-146. Maintainers have since agreed that 190 is the official number. This moves the file back and repairs every inbound reference in the same commit, so no link is broken at any point.

  • apps/0146.mdapps/0190.md, with the title line updated
  • Index entries in README.md, SUMMARY.md and apps/README.md
  • Inbound references in peer-to-peer/0169.md (2), apps/0168.md (2), apps/0210.md (14)
  • BRC-210's reference list reordered, since 190 now sorts after 169 rather than before 147

146 is left unallocated. That is deliberate rather than an oversight, but happy to renumber something into it if you would rather not have a gap.

It also fixes two dead links in a published standard

apps/0218.md links to ./0190.md in section 11 and again in its reference list. That path has returned 404 since #191 merged under a different number, so BRC-218 has been shipping two broken references. They resolve correctly once this lands, with no edit to BRC-218 itself.

Care taken

  • Replacements are anchored to 0146.md and BRC-146. A bare 0146 substitution would corrupt a SHA-256 in BRC-168's test vectors (...b1b75801462117... contains the string), and those bytes are untouched here.
  • Reference lists in all three edited documents verified still ascending, per the repository's ordering convention.

Separate finding, not touched by this PR

apps/0227.md cites "BRC-146" six times, but attributes to it the title Miner-Enforced Resale-Royalty Covenant Tokens (OP_PUSH_TX), which is BRC-226. Those are plain-text citations with no links, so they are unaffected by this rename, but they look like a typo for 226 and a naive find-and-replace would have made them worse. Left alone deliberately. Happy to fix in a follow-up if you agree with the reading.

🤖 Generated with Claude Code

Submitted as BRC-190 in bsv-blockchain#191 and merged as BRC-146. Maintainers have
since agreed 190 is the official number, so this moves the file back and
repairs every inbound reference.

apps/0146.md -> apps/0190.md with the title line updated, and the index
entries in README.md, SUMMARY.md and apps/README.md follow it.

Inbound references updated in peer-to-peer/0169.md (2), apps/0168.md (2)
and apps/0210.md (14). BRC-210's reference list is reordered, since 190
now sorts after 169 rather than before 147.

This also repairs two dead links in the published standard: apps/0218.md
section 11 and its reference list both point at ./0190.md, which has
404'd since bsv-blockchain#191 merged under a different number.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 23, 2026 16:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Renames the “Access Gates for Metanet Rooms” standard back to its agreed canonical number (BRC-190) and updates inbound references so existing links resolve correctly.

Changes:

  • Renamed the spec file to apps/0190.md and updated its title to BRC-190.
  • Updated inbound references in peer-to-peer/0169.md, apps/0168.md, and apps/0210.md to point to BRC-190.
  • Updated index entries in README.md, SUMMARY.md, and apps/README.md to reference apps/0190.md.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
SUMMARY.md Updates Apps index entry to point to apps/0190.md (needs reordering fix).
README.md Updates the main BRC index row to point to apps/0190.md (needs reordering fix).
peer-to-peer/0169.md Updates inline and reference-list links from BRC-146 to BRC-190.
apps/README.md Updates Apps index entry to point to apps/0190.md (needs reordering fix).
apps/0210.md Updates multiple textual references and the References section from BRC-146 to BRC-190.
apps/0190.md Updates the document title to BRC-190.
apps/0168.md Updates inline and reference-list links from BRC-146 to BRC-190.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread SUMMARY.md
Comment on lines 15 to 18
* [Registry-Free Typed Content Anchor with On-Chain Code Provenance](./apps/0145.md)
* [Access Gates for Metanet Rooms](./apps/0146.md)
* [Access Gates for Metanet Rooms](./apps/0190.md)
* [Verifiable Time Allocation](./apps/0168.md)
* [Derived Collectibles](./apps/0210.md)
Comment thread apps/README.md
Comment on lines 9 to 12
145 | [Registry-Free Typed Content Anchor with On-Chain Code Provenance](./0145.md)
146 | [Access Gates for Metanet Rooms](./0146.md)
190 | [Access Gates for Metanet Rooms](./0190.md)
168 | [Verifiable Time Allocation](./0168.md)
210 | [Derived Collectibles](./0210.md)
Comment thread README.md
Comment on lines 211 to 213
145 | [Registry-Free Typed Content Anchor with On-Chain Code Provenance](./apps/0145.md)
146 | [Access Gates for Metanet Rooms](./apps/0146.md)
190 | [Access Gates for Metanet Rooms](./apps/0190.md)
147 | [1Sat Ordinals Basket Profile for BRC-46 / BRC-100](./tokens/0147.md)
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.

2 participants