Skip to content

Remove the alpha label from the Solana integration page - #690

Merged
anxolin merged 3 commits into
mainfrom
solana-remove-alpha-label
Nov 15, 2026
Merged

anxolin merged 3 commits into
mainfrom
solana-remove-alpha-label

Conversation

@anxolin

@anxolin anxolin commented Nov 15, 2026 •

Copy link
Copy Markdown
Contributor

Description

Removes the alpha warning from the docs and points to the audits dir in the Solana program (will be publish here).

The Solana integration is no longer in alpha: Neodyme audited the
settlement program in October 2026. Replace the alpha caution with a
summary of the audit result and a link to the report.

Keep the part of the old warning that still holds — that staging and
production both run on mainnet-beta with real tokens — as an info note
in the environments section.
@vercel

vercel Bot commented Nov 15, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Nov 15, 2026 1:31pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Nov 15, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The Solana integration guide replaces an alpha-integration caution with an audit notice and adds information about the networks, tokens, order book, and settlement program used in staging and production.

Changes: Solana integration guidance

Layer / File(s) Summary
Audit and network guidance
docs/cow-protocol/integrate/solana.mdx
The guide links to the Neodyme audit report. It states that staging and production use mainnet-beta and real tokens, and that staging has a separate order book and settlement program.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: igorroncevic

Merge Risk: Minimal · up to ebb8c

This updates the Solana integration page to link to the audit report in place of the alpha warning, and it clarifies that staging runs on mainnet-beta with real tokens. Before publishing, confirm that the report PR has merged and that the audit covers the deployed program, so the audit claim and link are accurate. No code behavior changes.

Pre-merge checks: Passed 3 · Inconclusive 1

Failed Failed checks: 1 inconclusive

Check name Status Explanation Resolution
Description check Inconclusive The description is brief and states the purpose: it removes the alpha warning from the docs and points to the audits directory in the Solana program. It does not follow the template's required section… Add a Changes section with a checklist of the detailed edits. List the alpha caution removal, the Neodyme audit notice with its link, the new :::info callout in Choose an Environment, and the staging and production mainnet-beta note. Note t…
Passed Passed checks: 3 passed

Check name Status Explanation
Title check Passed The title says the alpha label is removed from the Solana integration page. The raw summary confirms the change replaces the alpha caution with an audit notice and an info callout. The title names the…
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.
Full details: Title check

Explanation

The title says the alpha label is removed from the Solana integration page. The raw summary confirms the change replaces the alpha caution with an audit notice and an info callout. The title names the main change, so it passes.

Full details: Description check

Explanation

The description is brief and states the purpose: it removes the alpha warning from the docs and points to the audits directory in the Solana program. It does not follow the template's required sections. The Changes section with a checklist of detailed changes is missing, and the Related Issues section is absent. The description does not list the audit link, the new info callout, or the dependency on cowprotocol/solana-programs#219 that the objectives mention. The core intent is clear, so the description is partly complete but lacks the template's structure and detail.

Resolution

Add a Changes section with a checklist of the detailed edits. List the alpha caution removal, the Neodyme audit notice with its link, the new :::info callout in Choose an Environment, and the staging and production mainnet-beta note. Note that the audit link depends on cowprotocol/solana-programs#219 merging first. Add the Related Issues section, or state that no issue is linked. Keep the existing Description, and explain briefly why the alpha label is being removed.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 66dfe998-2df5-4f66-b29b-f2dc11078e0f
Commits

Reviewing files that changed from the base of the PR and between cb225ed and ebb8cb9.

Files selected for processing: 1

  • docs/cow-protocol/integrate/solana.mdx

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

The report covers the findings; the page only needs to say the program
was audited and where to read it.
The report carries its own date; the page doesn't need to repeat it.
@anxolin
anxolin marked this pull request as ready for review November 15, 2026 13:32
@anxolin
anxolin requested a review from a team as a code owner November 15, 2026 13:32

@fedgiac fedgiac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. I didn't check whether there are more alpha warnings.

The Solana integration is in alpha, so program versions and APIs can change. Both staging and production use Solana mainnet-beta and real tokens; use a separate wallet with small amounts for testing.

:::
The settlement program was audited by [Neodyme](https://neodyme.io). Read the [audit report](https://github.com/cowprotocol/solana-programs/tree/main/audits).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@anxolin
anxolin merged commit a8f1d1e into main Nov 15, 2026
5 checks passed
@anxolin
anxolin deleted the solana-remove-alpha-label branch November 15, 2026 15:26
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 15, 2026

This branch was successfully deployed

1 active deployment
Preview — ebb8cb93 Deployed Nov 15, 2026 by vercel[bot]
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.

2 participants