chore(ci): consolidate test commands into justfiles and fix invocation#2228
Open
chore(ci): consolidate test commands into justfiles and fix invocation#2228
Conversation
- Add granular test targets to program-tests/justfile - Update programs.yml to use justfile targets instead of raw cargo commands - Add granular test targets to sdk-tests/justfile - Update sdk-tests.yml to use justfile targets - Add granular test targets to prover/server/justfile (including lean-install) - Update prover-test.yml to use justfile targets for all test types - Add test-compressed-token-e2e-ctoken-v2 target to js/justfile - Update js-v2.yml to use justfile command - Add comprehensive lint targets to main justfile - Update lint.yml to use justfile instead of direct script call Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Use :: separator for module::target syntax instead of space separator Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Refactor CI workflow to use Justfile for test runs
Consolidate CI test commands into Justfiles
Feb 2, 2026
Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Copilot
AI
changed the title
Consolidate CI test commands into Justfiles
chore(scripts): consolidate CI test commands into justfiles
Feb 2, 2026
sergeytimoshin
changed the title
Contributor
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (5)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
Comment |
Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Use double colon (::) separator for just submodule commands instead of space. This fixes test failures caused by incorrect command syntax. Fixed in workflows: - cli-v1.yml: just cli::build, just cli::test - cli-v2.yml: just cli::build, just cli::test - forester-tests.yml: just cli::build, just forester::* - js.yml: just cli::build, just js::* - js-v2.yml: just cli::build, just js::*, just sdk-tests::build-anchor-test - programs.yml: just cli::build - rust.yml: just cli::build, just program-libs::* - sdk-tests.yml: just cli::build, just sdk-libs::test Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Copilot
AI
changed the title
chore(scripts): consolidate CI test commands into justfile
fix: correct just submodule invocation syntax in workflows
Feb 2, 2026
The prover-test.yml workflow was failing because 'just' was not installed before attempting to run 'just prover::*' commands. Added the just installation step using extractions/setup-just@v2 to all test jobs in the workflow. Fixed jobs: - test-no-redis: Added just installation before running test targets - test-with-redis: Added just installation before running test targets - integration-test-lightweight: Added just installation - integration-test-lightweight-lazy: Added just installation - integration-test-full: Added just installation - lean-verification: Added just installation before lean-install Co-authored-by: sergeytimoshin <123843+sergeytimoshin@users.noreply.github.com>
Copilot
AI
changed the title
fix: correct just submodule invocation syntax in workflows
chore(ci): consolidate test commands into justfiles and fix invocation
Feb 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI workflows were running raw
cargo-test-sbf,go test, andpnpmcommands directly, creating drift between local development and CI environments.Changes
Added granular justfile targets
program-tests/justfile: 18 test targets (account-compression, system-cpi variants, compressed-token tests)sdk-tests/justfile: 17 test targets (native, anchor, pinocchio, token suites)prover/server/justfile: Addedlean-installfor formal verification setupjs/justfile: Added V2-specific test targetsjustfile: Consolidated lint functionality (dependencies, readmes, feature checks)Fixed workflow invocations
just cli build→just cli::build(8 workflows)justinstallation toprover-test.yml(6 jobs)Affected workflows
programs.yml,sdk-tests.yml,prover-test.yml: Now usejust <module>::<target>cli-v1.yml,cli-v2.yml,js.yml,js-v2.yml: Fixed CLI and JS test invocationsforester-tests.yml,rust.yml: Fixed module target syntaxlint.yml: Now usesjust lintinstead of direct script executionExample
Commands now have single source of truth in justfiles. CI executes identical commands to local development.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.