Skip to content

fix(ci): publish github package under org scope#4

Merged
Jess Sullivan (Jesssullivan) merged 2 commits into
mainfrom
codex/tin713-gh-packages-scope
Apr 28, 2026
Merged

fix(ci): publish github package under org scope#4
Jess Sullivan (Jesssullivan) merged 2 commits into
mainfrom
codex/tin713-gh-packages-scope

Conversation

@Jesssullivan

Copy link
Copy Markdown
Contributor

Fixes the GitHub Packages mirror scope from the public npm scope to the GitHub org scope.

The public npm artifact remains under @tummycrypt/*; this only changes the github_package_name input used by the reusable publish workflow so GitHub Packages publishes under @tinyland-inc/*.

Tracking: TIN-713

@greptile-apps

greptile-apps Bot commented Apr 28, 2026

Copy link
Copy Markdown

Greptile Summary

Corrects the GitHub Packages scope in both ci.yml and publish.yml by changing github_package_name from @tummycrypt/tinyland-activitypub to @tinyland-inc/tinyland-activitypub. The public npm artifact scope (@tummycrypt/*) is untouched; only the GitHub Packages mirror path is updated.

Confidence Score: 5/5

Safe to merge — a one-line scope rename in each workflow file with no logic changes.

Both files contain a single, consistent string substitution (@tummycrypt@tinyland-inc) in the github_package_name input. No logic, permissions, secrets handling, or publish conditions are altered. The CI job remains a dry run and the publish job already holds packages: write.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/ci.yml Updates github_package_name from @tummycrypt/tinyland-activitypub to @tinyland-inc/tinyland-activitypub; dry_run remains true so no actual publish occurs.
.github/workflows/publish.yml Same scope fix as ci.yml; packages: write permission is already present and NPM_TOKEN is explicitly forwarded, so the updated GitHub Packages target will publish correctly.

Sequence Diagram

sequenceDiagram
    participant GH as GitHub Actions
    participant CI as ci.yml (dry_run=true)
    participant PUB as publish.yml (on release)
    participant NPM as npm registry (@tummycrypt/*)
    participant GHP as GitHub Packages (@tinyland-inc/*)

    GH->>CI: push / PR to main
    CI->>GH: js-bazel-package workflow (dry run, no publish)

    GH->>PUB: release published / workflow_dispatch
    PUB->>NPM: publish @tummycrypt/tinyland-activitypub (NPM_TOKEN)
    PUB->>GHP: publish @tinyland-inc/tinyland-activitypub (GITHUB_TOKEN)
Loading

Reviews (1): Last reviewed commit: "fix(ci): publish github package under or..." | Re-trigger Greptile

@Jesssullivan Jess Sullivan (Jesssullivan) merged commit 52acb74 into main Apr 28, 2026
5 checks passed
@Jesssullivan Jess Sullivan (Jesssullivan) deleted the codex/tin713-gh-packages-scope branch April 28, 2026 21:59
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