Skip to content

ci: run tests on windows-latest#1997

Open
yottahmd wants to merge 59 commits intomainfrom
ci/windows-latest-test-matrix
Open

ci: run tests on windows-latest#1997
yottahmd wants to merge 59 commits intomainfrom
ci/windows-latest-test-matrix

Conversation

@yottahmd
Copy link
Copy Markdown
Collaborator

@yottahmd yottahmd commented Apr 12, 2026

Summary

  • run the existing CI test matrix on ubuntu-latest and windows-latest
  • fix Windows-specific test assumptions in the command, agent, and utility test suites
  • keep the Ubuntu coverage upload unchanged

Testing

  • go test ./internal/cmd -run 'Test(ServerCommand|CoordinatorCommand|SchedulerCommand|StartAllCommand|RunAIInstall|AIToolDetection|DetectAITargetsIncludesBothCodexDirectories)' -count=1
  • go test ./internal/agent -run 'Test(ResolvePath|ReadTool_Run|BashTool|SessionManager_RecordExternalMessage_UpdatesLastActivity|SessionManager_RecordHeartbeat)' -count=1
  • go test ./internal/service/coordinator -run TestLogHandler_LogFilePath -count=1
  • go test ./internal/cmn/buildenv -run TestPrepareAndLoad -count=1
  • go test ./internal/cmn/cmdutil -run TestShellQuote_ShellRoundTrip -count=1
  • GitHub Actions CI / Test on windows-latest

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 12, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9ba614df-9b4c-4108-a188-ad40a76bb794

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

A single line in the CI workflow configuration was uncommented to add windows-latest to the test job's platform matrix, enabling tests to run on both Ubuntu and Windows environments instead of Ubuntu only.

Changes

Cohort / File(s) Summary
CI Configuration
.github/workflows/ci.yaml
Uncommented windows-latest in the test matrix to expand CI job execution across Ubuntu and Windows platforms.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: run tests on windows-latest' directly matches the main change: uncommenting windows-latest in the CI test matrix to expand test execution to Windows.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/windows-latest-test-matrix

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.

@yottahmd yottahmd changed the title ci: run tests on windows-latest ci: add windows smoke CI job Apr 12, 2026
@yottahmd yottahmd changed the title ci: add windows smoke CI job ci: run tests on windows-latest Apr 12, 2026
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.

1 participant