Skip to content

Keep peer ranges in sync and dedupe Yarn package updates - #134

Merged
woksin merged 6 commits into
mainfrom
fix/110-peer-updates-and-yarn-dedupe
Oct 2, 2026
Merged

woksin merged 6 commits into
mainfrom
fix/110-peer-updates-and-yarn-dedupe

Conversation

@woksin

@woksin woksin commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Fixed

woksin added 5 commits October 1, 2026 23:52
Restore the unchanged ncu command so caller dependency-section policy is not overridden. Synchronize tracking peers only when the new pin satisfies the old range; warn instead of crossing compatibility boundaries.

Snapshot only the root and declared workspaces, tolerate BOMs and non-object sections, and retain symlink refusal. Cover the regressions while retaining Yarn dedupe and failure propagation.
@woksin woksin self-assigned this Oct 2, 2026
@woksin

woksin commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

Local checks passed on follow-up commit e4d925722884090bc661103e2be6666456ce3670:

  • python3 -B .github/scripts/tests/update-packages.test.py — 76 tests passed; the initial 120-second attempt timed out, and the bounded 300-second rerun completed in 156 seconds.
  • python3 -B .github/scripts/tests/sync-package-update-peers.test.py — 26 tests passed, including symlinked parents, skipped-match notices, annotation escaping, and continued refusal of symlinked manifests. The update suite also enforces byte equality of the embedded helper and standalone script.
  • python3 -B .github/scripts/tests/actions-usage-report.test.py — 34 tests passed.
  • python3 -B .github/scripts/tests/bootstrap-package-update-safety.test.py — 5 tests passed.
  • python3 -B .github/scripts/tests/bootstrap-schedules.test.py — 3 tests passed.
  • python3 -B .github/scripts/tests/cleanup-pr-artifacts.test.py — 35 tests passed.
  • node --test .github/scripts/tests/verify-release-intent.spec.mjs — 10 tests passed.
  • node --test .github/scripts/tests/verify-release-notes.spec.mjs — 63 tests passed.
  • node --check .github/scripts/update-ai-profile-subscription.mjs and node --test .github/scripts/update-ai-profile-subscription*.spec.mjs — syntax check and 18 tests passed.
  • actionlint .github/workflows/update-packages.yml and actionlint .github/workflows/verify-package-updates.yml — passed.
  • git diff --check — passed. Fetch/rebase confirmed the branch was already up to date.

These are the repository's native script gates; there is no application solution/build or separate integration lane. Local execution used macOS and Node 26.8.1. CI's Ubuntu environment and Node 22/24 configurations are CI-only because those Node versions are not installed locally. No live dependency-update or organization-wide write workflow was invoked.

Release intent is no-release: Workflows has no releases or tags, generated callers reference its workflows at @main, and the publish template only handles Source/** changes or explicit manual dispatch, not this change.

The cross-provider review workflow (Opus 5.5 + GPT-6 Astra) passed on the pre-follow-up diff (round 4), with the confirmed minor finding addressed by e4d9257. The orchestrator will verify that follow-up before creating merge-ok. Merge remains blocked pending that authorization.

@woksin
woksin merged commit 2658175 into main Oct 2, 2026
@woksin
woksin deleted the fix/110-peer-updates-and-yarn-dedupe branch October 2, 2026 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant