Sign up
View stargazersSign up

chore(lint): disable unicorn/prefer-string-starts-ends-with#25045

merged1
lint-rules/unicorn-prefer-set-has
lint-rules/unicorn-prefer-string-starts-ends-with
opened5d agobyJJulius Marminge
1 files changed
View on GitHubTwo-way sync enabledPR last imported 4d ago

Anchored regular expressions are allowed. This rule is explicitly off, so these allowed patterns no longer emit advisory diagnostics.

This PR changes only this rule and its existing call sites. Each changed rule has its own PR.

Stack validation: rebased onto b76a110. The earlier 234-file test run passed, and the cancellation regression test passes after its local exception. Rebased format, lint, Knip, and all 34 affected-package typecheck tasks pass. The final local test rerun and fresh CI are pending.

Model: GPT-6.1 Sol. Harness: Codex.

Checks

23 successful · 13 skipped
All checks passed
862647c
Ggithub-actionsBot5d ago
Synced from GitHub

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire—5.0 KiB—6.8 KiB✅
CodexThread snapshot wire—3.8 KiB—4.9 KiB✅
CodexLive turn WebSocket wire—1.2 KiB—2.0 KiB✅
CodexLive turn WebSocket decoded—20.9 KiB—29.3 KiB✅
CodexLive turn messages—2—8✅
ClaudeTotal thread wire—5.0 KiB—6.8 KiB✅
ClaudeThread snapshot wire—3.8 KiB—4.9 KiB✅
ClaudeLive turn WebSocket wire—1.2 KiB—2.0 KiB✅
ClaudeLive turn WebSocket decoded—21.2 KiB—29.3 KiB✅
ClaudeLive turn messages—2—8✅

Baseline: unavailable · PR result: 862647c · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

MmacroscopeappBot5d ago
Synced from GitHub

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — This one-line change only alters lint behavior by disabling unicorn/prefer-string-starts-ends-with and does not affect product runtime behavior. Because it adds a configuration-level static-analysis suppression, human review is required.

Notes:

  • Diff unchanged. Approvability was decided on eligibility alone.

No code changes detected at 862647c. Prior analysis still applies.

You can add or adjust custom eligibility rules. Learn more.

CcoderabbitaiBot5d ago
Synced from GitHub

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

External image Run configuration
External image Commits

Reviewing files that changed from the base of the PR and between 36abb0e4663c665f569b1cf891e3f7216b2e4256 and 862647c9edb73171ab1de07252e1cf7161715fe1.

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

No actionable comments were generated in the recent review.

External image Recent review info
External image Run configuration
External image Commits

Reviewing files that changed from the base of the PR and between 0aa7721979ee88ecf6e8b0648acdb1691636383e and 36abb0e4663c665f569b1cf891e3f7216b2e4256.

External image Files selected for processing: 1
  • vite.config.ts

Limit details: You’ve used all 10 included reviews currently available.

Walkthrough

The shared Vite+ configuration now disables the unicorn/prefer-string-starts-ends-with lint rule.

Changes: Lint configuration

Layer / File(s)Summary
Set the Unicorn rule
vite.config.ts		The shared Vite+ configuration sets unicorn/prefer-string-starts-ends-with to off.

Priority: External image Low

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

Change: Other

Suggested reviewers: macroscopeapp

Merge Risk: External image Minimal · up to 36abb

This lint configuration change is safe to merge.

Pre-merge checks: Passed 4
Check nameStatusExplanation
Title checkPassedThe title clearly identifies the lint rule that the pull request disables. It matches the primary change.
Description checkPassedThe description explains the problem, the focused change, and verification results. It does not use the required section headings or provide explicit issue or maintainer approval details, but it is mo…
Linked Issues checkPassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes checkPassedCheck skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description explains the problem, the focused change, and verification results. It does not use the required section headings or provide explicit issue or maintainer approval details, but it is mostly complete for this small configuration change.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Added 1 commit

  • chore(lint): disable unicorn/prefer-string-starts-ends-with862647c