Skip to content

Remove Argos leftovers - #20

Merged
silveltman merged 1 commit into
mainfrom
chore/remove-argos-leftovers
Oct 1, 2026
Merged

silveltman merged 1 commit into
mainfrom
chore/remove-argos-leftovers

Conversation

@silveltman

@silveltman silveltman commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Argos is no longer used anywhere in the organization. This removes what was left in sitex:

  • @argos-ci/cli and @argos-ci/playwright from the catalog in pnpm-workspace.yaml.
  • @playwright/test from the same catalog. Nothing else in the repo uses it.

No package.json, workflow or config referenced these entries, so pnpm-lock.yaml does not change and the build output stays the same. pnpm check and pnpm build pass.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated package management configuration. The CLI and browser-testing packages are no longer included in the shared workspace catalog.

Argos is no longer used. Drop @argos-ci/cli, @argos-ci/playwright and
@playwright/test from the pnpm catalog. No package referenced them.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@silveltman
silveltman marked this pull request as ready for review October 1, 2026 13:56
@coderabbitai

coderabbitai Bot commented Oct 1, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8bbc2640-3e94-4c12-95de-2d4f4290a5ed

📥 Commits

Reviewing files that changed from the base of the PR and between 99d3165 and aa2906e.

📒 Files selected for processing (1)
  • pnpm-workspace.yaml
💤 Files with no reviewable changes (1)
  • pnpm-workspace.yaml

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


📝 Walkthrough

Walkthrough

The workspace catalog no longer lists @argos-ci/cli, @argos-ci/playwright, or @playwright/test.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to aa290

The cleanup removes unused dependency catalog entries without affecting existing consumers. No actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the removal of unused Argos-related entries from the workspace catalog.
Description check ✅ Passed The description accurately describes the removed catalog entries and reports the validation results.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@silveltman
silveltman merged commit 014d38d into main Oct 1, 2026
4 checks passed
@silveltman
silveltman deleted the chore/remove-argos-leftovers branch October 1, 2026 14:24
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