Skip to content

Update token.md - Include COW in BNB address#592

Merged
alfetopito merged 3 commits intomainfrom
blacktieaccountant-patch-2
Feb 6, 2026
Merged

Update token.md - Include COW in BNB address#592
alfetopito merged 3 commits intomainfrom
blacktieaccountant-patch-2

Conversation

@blacktieaccountant
Copy link
Contributor

@blacktieaccountant blacktieaccountant commented Feb 5, 2026

Add address to BNB version of COW.
Did not include footnote as the bridged token does use stargate and not a native bridge.

Thread with context and validation of token address: https://cowservices.slack.com/archives/C06G1E3AU77/p1758625335858369

Description

Changes

  • ...
  • ...

Summary by CodeRabbit

  • Documentation
    • Updated the COW token entry for BNB to include the correct token contract address and a link to the blockchain explorer.
    • Added a new footnote explaining Stargate-bridged contracts; existing bridged-token footnote remains unchanged.

Add address to BNB version of COW.
Did not include footnote as the bridged token does use stargate and not a native bridge.

Thread with context and validation of token address:
https://cowservices.slack.com/archives/C06G1E3AU77/p1758625335858369
@blacktieaccountant blacktieaccountant requested a review from a team as a code owner February 5, 2026 11:33
@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Feb 6, 2026 4:27pm

Request Review

@blacktieaccountant blacktieaccountant self-assigned this Feb 5, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

BNB entry in the COW token addresses table (docs/governance/token.md) was changed from "N/A" to a specific BNB token contract address with a BSCScan verification link and a new footnote [^stargateBridge] describing Stargate-bridged contracts.

Changes

Cohort / File(s) Summary
Token Documentation
docs/governance/token.md
Replaced BNB token address value "N/A" with a concrete BNB token contract address, added a BSCScan link, and introduced a new footnote [^stargateBridge] describing Stargate-bridged contracts. Existing bridged-token footnote block left unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • elena-zh

Poem

🐇 I hopped through tokens, found a clue,
Swapped "N/A" for a link that's true,
A Stargate note tucked in with care,
Docs now sparkle — a hop in the air,
Crunching carrots, I cheer and stare.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Update token.md - Include COW in BNB address' clearly and specifically describes the main change in the pull request—adding the COW token address for the BNB chain.
Description check ✅ Passed The description explains the change and provides context via a Slack link, but the Description and Changes sections from the template remain as unfilled placeholder comments rather than being completed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch blacktieaccountant-patch-2

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/governance/token.md`:
- Line 22: The BNB token address row in the markdown table uses a plain link but
should match the other entries by wrapping the address in backticks; update the
BNB row (the line containing "BNB" and the link to bscscan token
0x5bfdaa3f7c28b9994b56135403bf1acea02595b0) so the linked address is enclosed in
backticks (e.g., `0x5bf...95b0`) while preserving the existing markdown link
target and spacing for consistency and copy/paste parity with the other address
entries.

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details
{"name":"HttpError","status":401,"request":{"method":"PATCH","url":"https://api.github.com/repos/cowprotocol/docs/issues/comments/3853044021","headers":{"accept":"application/vnd.github.v3+json","user-agent":"octokit.js/0.0.0-development octokit-core.js/7.0.6 Node.js/24","authorization":"token [REDACTED]","content-type":"application/json; charset=utf-8"},"body":{"body":"<!-- This is an auto-generated comment: summarize by coderabbit.ai -->\n<!-- This is an auto-generated comment: review in progress by coderabbit.ai -->\n\n> [!NOTE]\n> Currently processing new changes in this PR. This may take a few minutes, please wait...\n> \n> \n> \n> ```ascii\n>  ________________________________________\n> < Your docs say 'TBD'. I'm here to 'Do'. >\n>  ----------------------------------------\n>   \\\n>    \\   \\\n>         \\ /\\\n>         ( )\n>       .( o ).\n> ```\n> \n> <sub>✏️ Tip: You can disable in-progress messages and the fortune message in your review settings.</sub>\n\n<!-- end of auto-generated comment: review in progress by coderabbit.ai -->\n<!-- usage_tips_start -->\n\n> [!TIP]\n> <details>\n> <summary>CodeRabbit can generate a title for your PR based on the changes.</summary>\n> \n> Add `@coderabbitai` placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the `reviews.auto_title_placeholder` setting.\n> \n> </details>\n\n<!-- usage_tips_end -->\n<!-- walkthrough_start -->\n\n<details>\n<summary>📝 Walkthrough</summary>\n\n## Walkthrough\n\nThe BNB token contract address in the COW token documentation was updated from \"N/A\" to a specific contract address with a BSCScan verification link in the token contract addresses reference table.\n\n## Changes\n\n|Cohort / File(s)|Summary|\n|---|---|\n|**Token Documentation** <br> `docs/governance/token.md`|Updated BNB token contract address entry from \"N/A\" to a specific address with BSCScan link.|\n\n## Estimated code review effort\n\n🎯 1 (Trivial) | ⏱️ ~3 minutes\n\n## Possibly related PRs\n\n- cowprotocol/docs#538 — Previously added the BNB entry as \"N/A\"; this PR provides the actual contract address.\n- cowprotocol/docs#495 — Also modifies token documentation within the same file for additional governance content updates.\n\n## Suggested reviewers\n\n- pretf00d\n- fhenneke\n\n## Poem\n\n> 🐰 A token address found at last,  \n> From \"N/A\" to verified past,  \n> On BSCScan it gleams so bright,  \n> The BNB chain now set right! ✨\n\n</details>\n\n<!-- walkthrough_end -->\n\n<!-- pre_merge_checks_walkthrough_start -->\n\n<details>\n<summary>🚥 Pre-merge checks | ✅ 2 | ❌ 1</summary>\n\n<details>\n<summary>❌ Failed checks (1 inconclusive)</summary>\n\n|     Check name    | Status         | Explanation                                                                                                                                              | Resolution                                                                                                                  |\n| :---------------: | :------------- | :------------------------------------------------------------------------------------------------------------------------------------------------------- | :-------------------------------------------------------------------------------------------------------------------------- |\n| Description check | ❓ Inconclusive | The description provides context for the change and references supporting documentation, but the required template sections are not properly filled out. | Complete the 'Description' and 'Changes' sections according to the template to provide clearer documentation for reviewers. |\n\n</details>\n<details>\n<summary>✅ Passed checks (2 passed)</summary>\n\n|     Check name     | Status   | Explanation                                                                                                             |\n| :----------------: | :------- | :---------------------------------------------------------------------------------------------------------------------- |\n|     Title check    | ✅ Passed | The title clearly and accurately describes the main change: adding the COW token BNB address to token.md documentation. |\n| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.              |\n\n</details>\n\n<sub>✏️ Tip: You can configure your own custom pre-merge checks in the settings.</sub>\n\n</details>\n\n<!-- pre_merge_checks_walkthrough_end -->\n\n<!-- finishing_touch_checkbox_start -->\n\n<details>\n<summary>✨ Finishing touches</summary>\n\n<details>\n<summary>🧪 Generate unit tests (beta)</summary>\n\n- [ ] <!-- {\"checkboxId\": \"f47ac10b-58cc-4372-a567-0e02b2c3d479\", \"radioGroupId\": \"utg-output-choice-group-unknown_comment_id\"} -->   Create PR with unit tests\n- [ ] <!-- {\"checkboxId\": \"07f1e7d6-8a8e-4e23-9900-8731c2c87f58\", \"radioGroupId\": \"utg-output-choice-group-unknown_comment_id\"} -->   Post copyable unit tests in a comment\n- [ ] <!-- {\"checkboxId\": \"6ba7b810-9dad-11d1-80b4-00c04fd430c8\", \"radioGroupId\": \"utg-output-choice-group-unknown_comment_id\"} -->   Commit unit tests in branch `blacktieaccountant-patch-2`\n\n</details>\n\n</details>\n\n<!-- finishing_touch_checkbox_end -->\n\n<!-- tips_start -->\n\n---\n\nThanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=cowprotocol/docs&utm_content=592)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.\n\n<details>\n<summary>❤️ Share</summary>\n\n- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai)\n- [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai)\n- [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai)\n- [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)\n\n</details>\n\n<sub>Comment `@coderabbitai help` to get the list of available commands and usage tips.</sub>\n\n<!-- tips_end -->\n\n<!-- internal state start -->\n\n\n<!-- DwQgtGAEAqAWCWBnSTIEMB26CuAXA9mAOYCmGJATmriQCaQDG+Ats2bgFyQAOFk+AIwBWJBrngA3EsgEBPRvlqU0AgfFwA6NPEgQAfACgjoCEYDEZyAAUASpETZWaCrKPR1AGxJcAqt1rUJJAEANZkGsz0UACSGAwe2EqQAMIA8gDqKFgAQgBy2ei0tBTSyJAGuY4ClFwArACcAEyQ5T42ADJcsLi43IgcAPQDROqw2AIaTMwDTADuvPgETB4DtPgMiAPc2B4rDc2tiDWQAh5oDCHiJOdM2Bi4mLgtBgDK+NgUDEECVHGwXKdzpd4NcGLd7o8wNxqAxYGADgZoM5SE8fphYVxmNosOUXg9cNh+vxuGRnskSoF6NQuI0AAyNABsYHpzNq0AAjOyOABmbkcAAsAHYAFrPGwkCQg2aUIkYfDkSAeJA0ejldoqEgeWXyoJKxAqowAEWkDAo8G44nlkAAFAAzc64ZDyjyyACUHAMUAAgkVkLhYEE8tkwLDsYViqVILb8Hx/UE0plQqSCMF8GEMBFaBpPTAA9Y7GtpJA5U80EV0FH8IsS0ESAAPbhneAYZtEE5m2gjDBt6oMNCEoJx9vwTt0VPpyAD5B45GBSBUOOx2CYCsYaiSb4d0jZ72QF5nC7BWAU+gSNBKgLibtHwdp0lliOIZCoEq2yhkL60Lo9PqDGb4WYjgoSUvkQDREAPEJJhYAZnFhDdNmSWkGQAcXZABRbkvR8QVBS2dlBVqAAOBlGlqXliOI7kGXqIxcnwcN1HgeVz0YZdu2kAAaedNUpFAn2wLj+D4EpJRIaUKGQZwgkQEkGHgW0QXoZsbx4Ch8ElJR6FsSAlEQU1zUtDMjH0YxwCgMh6HwW0cAIYgyGUFUFFYdguF4fhhFEcQpBkeQmCUKhVHULQdFMkwoDgVBUBXfs7NIcgFzHKY2HuLgqFmexHCxFwTj8xRlCCzRtF0MBDDM0wDDWDZhg0yg1ziEgBiTDNIg9AAiDqDAsSAvWieyEr4hwnBy6y2MwUhEDcPMg0gdgcpUocE3HUkmHuKgxHDEonyLB5TiCWY0GQbB/D4211OYSA2tyAYvTa1MK1k0QFPgBhIBm5rNsjWZRgrJUMBCe7sheZIXj7DNIHo/hF0VZsiwkoJmEUZ66AAbnQXYoYDWM7ywOaQWQEosRUu5Qw4rMjHMSwvQ8GgF2YjA/QYoclHiZx13lJ0bPrbgYycmMeHGJVXvYJjpBMiGGO2U4XoGbneboMARdweREHgIg1wJEoxo45AlBoMQ6GzLrLAAWUwBTpCeAAxeAvB6tcXQAL0oIx2lh5BSdIL9IAAanZAYwHZIx0P1eAsSc/yglEqVZttaMKE4SATboeBHAMDq2pM0ryosjArJs2LCHixykpYFLE/SzLhvkOQFAClQ1CK0Ls/C5zmHUAB9EdEA76PxLoDv9WcJ5TIMVuSG5WgiO5AR6SIhhJ6UfkGRIZeBCI9kBG5epaltOkBAYWh6n5epp9pXkSrK1vks77ve4lKUB8sy+x/MtSSA7tgKFIDvYVEEIe5DwTi/AwABvAwLQ2pIFsNkDw6wwi0GSGXdgVh8D6joG1Lg9otQkE4hAy6iBYDvA8LQWB8DbCYKjOeI4eDIFIFSFICgHYlAYEodgmh+C2q0BHDYO4hp1h4jNN2RAyQAwXEobgCgglaGXW4bQXhGB3C4C8KI/+EipG4M4XIhRxp9JmgtPTVR4iuCSOkZwv6CDogCWkIIyhHUZFtTOPqIxIRxQOBpogShABtfBLRwEtACZdP+FxchoDYHYpRdtgkhDajIwJbUh4Ek8SYjRcSAltW5mcDW9MIl5nEMooI8RrgUBdOgPO6AwQfECKUvSBlqh+jzETLAntvCMWvItDIy0cj5E+k+e6zVMy6XWI4dg7MMyxN8ekxGSg7EHQoC2bsEzAmQJKKtRSRAPgkDYdQzRyzLoxjVs2c8LjQnhK4G1fJXhM6BIAL5pP8cstq0TTlbPObogyBirTRKWY8xJhJtk4LSZAzJmAxm5KCLU/RRk1IaRHEWVaNA6xPHjqpFpZT6CvnfA1ZADhuA8wTq2IZDARkQiMtxAQeBVIlAAI7YHgCUegNBmCNjnEcMQ9MpJaxLDCkkJT5CKV2GOd4mgfnxK2vgBIRk7FIOZV4GgqkADk7yoX0wVeiyACrRHjWkGqtlRkpJghjNw9pTM8kkFlXOFMCxNKFK8NJPgVUSX4nppWESD9+6SQ0KKqZ+VZnOAWUQb1KzRDynWZs9RZi9ltQOV2Y5YiQgvLsZCwyOTJl3MmQ8+JzywmvMuvwjYkjCVIMYWgUgQaCH4n+SkyNjyQXZPlHYyGtoSb6srHcZSWAhwtPoAKnaDEJTnmwHOKq+ohFtiYCW7ce4Qjmm4ISkdhbrwTuUKQNi/8vVAsuuKyVOTznpDNPKhdY7kAoqHM2uIrb25PkJXGC6KZEDrkQLaeQXbapUFXf6LaRCSEbsmZA6Zua2pzIDeW6NZpY0eBOTmpN6xR2tk8Wm/BABdBxTjcC2GVSmht5zaCCm5PycitAGCCgEIKdk/JuQMCIvyBgDIBC0FqAyBkgpT61FoI0AjRFaRkaULSZeDBGhNFqPyIijHhOCjIradkjQwQMGk1s1Dh10M2EiYB7eAn6h4a4wyIiRFcL8jfNyW0/IKMsfqHHWgdGWOETY6IOTy9Gjsm5OyWoaBj6idtPUbkgp+QCDQGgZjtIBA/K4bBxdRBi0rpILEWmDsZw0EoaAm5BhkuvwgO/T+lAf7RJ7s/fQQA -->\n\n<!-- internal state end -->"},"request":{"retryCount":1}},"response":{"url":"https://api.github.com/repos/cowprotocol/docs/issues/comments/3853044021","status":401,"headers":{"access-control-allow-origin":"*","access-control-expose-headers":"ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset","connection":"close","content-security-policy":"default-src 'none'","content-type":"application/json; charset=utf-8","date":"Thu, 05 Feb 2026 12:00:13 GMT","referrer-policy":"origin-when-cross-origin, strict-origin-when-cross-origin","server":"github.com","strict-transport-security":"max-age=31536000; includeSubdomains; preload","vary":"Accept-Encoding, Accept, X-Requested-With","x-content-type-options":"nosniff","x-frame-options":"deny","x-github-media-type":"github.v3; format=json","x-github-request-id":"8025:F8AF3:4013BA:117B64A:6984864D","x-xss-protection":"0"},"data":{"message":"Requires authentication","documentation_url":"https://docs.github.com/rest","status":"401"}}}

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/governance/token.md`:
- Line 22: The BNB table row contains an unofficial token address
`0x5bfdaa3f7c28b9994b56135403bf1acea02595b0`; revert the BNB entry in
docs/governance/token.md so the BNB row reads "N/A" instead of that address
(i.e., replace the table cell content for the BNB row that currently contains
the address with "N/A") to match official CoW Protocol deployments.

@alfetopito alfetopito enabled auto-merge (squash) February 6, 2026 16:28
@alfetopito alfetopito disabled auto-merge February 6, 2026 16:28
@alfetopito alfetopito merged commit ced3522 into main Feb 6, 2026
4 of 5 checks passed
@alfetopito alfetopito deleted the blacktieaccountant-patch-2 branch February 6, 2026 16:28
@github-actions github-actions bot locked and limited conversation to collaborators Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants