Skip to content

Retire package-version bump and synchronization automation - #136

Merged
Eduardo Villalpando Mello (edvilme) merged 3 commits into
mainfrom
copilot/retire-package-version-bump
Nov 11, 2026
Merged

Eduardo Villalpando Mello (edvilme) merged 3 commits into
mainfrom
copilot/retire-package-version-bump

Conversation

Copilot AI commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Consumers identify compatibility by pinned Git revision, making synchronized npm/Python version bookkeeping unnecessary. Package metadata remains fixed for local build, installation, and diagnostics.

  • Automation cleanup

    • Remove weekly patch-bump workflow.
    • Remove duplicate manifest-version checks.
    • Delete scripts/version/ and root VERSION.
  • Metadata behavior

    • Preserve npm and Python manifest versions, lockfile metadata, and private-package safeguards.
    • Preserve the Python server’s installed-package diagnostic version.
  • Documentation

    • Define the pinned source commit as the compatibility identity.
    • Describe optional GitHub tags/releases as informational milestones.
    • Record S4 artifact-provenance enforcement as follow-up work.
  • Consumer and repository coordination

    • Black still uses published npm package ^0.8.2 alongside the Python submodule; its source migration remains separate.
    • Maintainers must verify that deleted check contexts are removed from protected-branch rules.

Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>
@edvilme Eduardo Villalpando Mello (edvilme) added the debt Code Cleanup, Refactorings and Repo health. Not feature related label Nov 11, 2026
Copilot AI changed the title [WIP] Retire package-version bump and synchronization automation Retire package-version bump and synchronization automation Nov 11, 2026
@edvilme
Eduardo Villalpando Mello (edvilme) marked this pull request as ready for review November 11, 2026 23:19
@rchiodo

Copy link
Copy Markdown
Contributor

🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

CONTRIBUTING.md incorrectly presents same-revision source builds as universal, despite Black still using a packaged TypeScript runtime and requiring a separate B0 migration.

1 open issue
What changed in this PR

Retires package-version bookkeeping in favor of pinned Git revisions as the compatibility identity, while leaving package metadata and Python diagnostic version reporting unchanged.

Changes:

  • Removes the weekly patch-bump workflow and both version-consistency checks.
  • Deletes the root VERSION file and version-management helpers.
  • Documents commit-based adoption, optional milestone releases, and deferred S4 artifact-provenance enforcement.
File Description
VERSION Removes the shared version source.
scripts/​version/​validate.py Removes manifest-version validation.
scripts/​version/​sync.py Removes manifest-version synchronization.
scripts/​version/​bump_patch.py Removes patch-bump automation.
scripts/​version/​__init__.py Removes shared version-management utilities.
CONTRIBUTING.md Replaces release bookkeeping instructions with source-adoption guidance.
.github/​workflows/​version-check.yml Removes the version-consistency check.
.github/​workflows/​pr-version.yml Removes the duplicate version-match check.
.github/​workflows/​auto-patch-bump.yml Removes scheduled and manual patch-bump automation.

Review effort: Balanced

Comment thread CONTRIBUTING.md Resolved
@edvilme

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

…e-version-bump

# Conflicts:
#	CONTRIBUTING.md

Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>

Copilot AI commented Nov 11, 2026

Copy link
Copy Markdown
Contributor Author

Copilot resolve the merge conflicts in this pull request

Merged main and resolved the CONTRIBUTING.md conflict in merge commit 754b40b. CodeQL found no alerts; tests could not run because pytest and npm dependencies are unavailable.

This comment was marked as low quality.

@rchiodo

Copy link
Copy Markdown
Contributor

Reviewed commit: 754b40ba811a009c144aa9e4fd772c3eb218ee92

Result: 🔴 could-not-verify

Verification details

Verification: Automated verification is best effort. The PR author remains responsible for testing.

Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine &#x27;pyrx-automation&#x27; is not running.

Test runs: none recorded.

@rchiodo Rich Chiodo (rchiodo) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved via Review Center.

@rchiodo Rich Chiodo (rchiodo) added the review-auto:approved Automated review: no blocking findings (approval posted). label Nov 11, 2026
@edvilme
Eduardo Villalpando Mello (edvilme) merged commit c6cd709 into main Nov 11, 2026
19 checks passed
@edvilme
Eduardo Villalpando Mello (edvilme) deleted the copilot/retire-package-version-bump branch November 11, 2026 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Code Cleanup, Refactorings and Repo health. Not feature related review-auto:approved Automated review: no blocking findings (approval posted).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Retire package-version bump and synchronization automation

4 participants