Skip to content

feat: anchor v1 migration skill#35

Open
tiago18c wants to merge 3 commits intosolana-foundation:mainfrom
tiago18c:anchor_migration_skill
Open

feat: anchor v1 migration skill#35
tiago18c wants to merge 3 commits intosolana-foundation:mainfrom
tiago18c:anchor_migration_skill

Conversation

@tiago18c
Copy link
Copy Markdown
Contributor

@tiago18c tiago18c commented Mar 12, 2026

Added a new skill for anchor migration from v0.32 to v1.

CI — surfpool must be installed explicitly:
```yaml
- name: Install surfpool
run: cargo install surfpool --version <pinned-version>
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.

Surfpool isn't available via cargo install.

This action is what anchor uses to install surfpool: https://github.com/txtx/anchor/blob/bc92c65b2f66e8e20110570f639d3be1292890dd/.github/actions/setup-surfpool/action.yaml

I will add to linear an action to make surfpool easier to use in CI though, this is too much work for the user.

@lgalabru
Copy link
Copy Markdown
Collaborator

@tiago18c this PR is now conflicting with main, would you have some cycles to rebase that branch? thanks!

@tiago18c
Copy link
Copy Markdown
Contributor Author

@lgalabru I'm moving this to draft, once we have the anchor RC out I'll finalize the PR as I want to add some other changes that are currently being finalized

@tiago18c tiago18c marked this pull request as draft March 19, 2026 18:05
@tiago18c tiago18c force-pushed the anchor_migration_skill branch from 071cbfa to 63fb766 Compare March 26, 2026 16:43
@tiago18c tiago18c marked this pull request as ready for review March 26, 2026 16:46
@tiago18c tiago18c requested a review from MicaiahReid March 26, 2026 16:47
3. **`anchor build`** — confirms Rust is clean.
4. **Update TS** — rename package imports, rerun `yarn install` / `npm install`.
5. **Run tests** — `anchor test` (surfpool) or `anchor test -- --features some-feature`.
6. **Just before deploying** — with the v0.32 CLI still installed, close legacy IDL accounts on every cluster (see §5). Upgrading the CLI or deploying the v1 binary first makes this impossible.
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.

I'm a bit confused by this directive after point 0. Step 0 says to upgrade anchor CLI, this is saying it should still be 0.32? Should this step (closing legacy IDL) actually be step 0?


# Or configure surfpool
[tooling.surfpool]
logs = false
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.

I don't believe logs is valid here. There's log_level, which defaults to none. Options are "trace", "debug", "info", "warn", "error"

Rmrifa

This comment was marked as off-topic.

@Rmrifa
Copy link
Copy Markdown

Rmrifa commented Mar 28, 2026

1ed6ca6

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.

4 participants