Skip to content

fix: point the counts generator at nself-org/bundles - #98

Merged
acamarata merged 1 commit into
mainfrom
fix/bundles-rename-counts-generator
Sep 17, 2026
Merged

acamarata merged 1 commit into
mainfrom
fix/bundles-rename-counts-generator

Conversation

@acamarata

Copy link
Copy Markdown
Contributor

The licensed-plugin repo was renamed plugins-pro -> bundles (ADR-P6-01), so counts.json was labelling its own source with a repo name that no longer exists.

The sibling-checkout lookup now prefers ../bundles and falls back to ../plugins-pro. A clone made before the rename still has the old directory name on disk and reaches GitHub through the redirect; hard-failing on those would break every existing dev machine and CI checkout for no gain.

counts.json regenerated with --origin (not hand-edited — it carries a DO-NOT-HAND-EDIT marker). Counts are unchanged: 129/127 free, 46/46 licensed, 171 advertised. Only sources.pro.repo and the two source blob SHAs move, confirming the rename did not disturb the registries.

The vendored copy in cli is updated in nself-org/cli#431.

The licensed-plugin repo was renamed plugins-pro -> bundles (ADR-P6-01),
so the artifact was labelling its own source with a name that no longer
exists.

The sibling-checkout lookup now prefers ../bundles and falls back to
../plugins-pro, because a clone made before the rename still has the old
directory name on disk and reaches GitHub through the redirect. Failing
on a pre-rename checkout would have broken every existing dev machine.

counts.json regenerated with --origin. Counts are unchanged: 129/127
free, 46/46 licensed, 171 advertised. Only sources.pro.repo and the two
source blob SHAs move.
@acamarata
acamarata merged commit 7cf0e3e into main Sep 17, 2026
29 checks passed
@acamarata
acamarata deleted the fix/bundles-rename-counts-generator branch September 17, 2026 11:18
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