Skip to content

fix(release): repair version drafting and align workflow naming - #37

Merged
aryasaatvik merged 4 commits into
mainfrom
fix/release-workflow-naming
Sep 19, 2026
Merged

aryasaatvik merged 4 commits into
mainfrom
fix/release-workflow-naming

Conversation

@aryasaatvik

Copy link
Copy Markdown
Member

Summary

The nightly/manual prepare release workflow failed because Tegami resolved every samva specifier to the private root package named samva, rewriting examples and publishable peers to ^0.0.0 and breaking bun install. This renames the private root and aligns the workflow files, jobs, and steps to the platform repo's lowercase convention.

Changes

  • Rename the private root package samva → @samva/integrations and ignore it in the Tegami release graph. bun run tegami version now completes and writes the publish lock.
  • Rename tegami-pr.yml → release-plan.yml and tegami-pr-comment.yml → release-plan-comment.yml.
  • Lowercase every workflow, job, and step name; rename the preview artifact to release-plan.
  • Update the workflow reference in docs/releases.md.

Review notes

  • Reproduced the original failure locally (samva@^0.0.0 failed to resolve after Tegami rewrote the samva specifiers) and confirmed the fix reaches Package versions updated → Publish lock written.
  • Verified bun install, bun run test (89), bun run release:packages:check, bun run validate:agent-plugin, format:check, lint, and Bun.YAML.parse over all five workflow files.
  • Direct push to main was rejected by the repository ruleset requiring a pull request, so this lands as a PR.
  • After merge, the prepare release workflow (nightly 13:00 UTC or manual dispatch) will draft the Version Packages PR. release-plan-comment.yml triggers on the release plan workflow name, which the renamed workflow now sets.

…ewrite

Tegami resolved every samva specifier to the private root package named samva, rewriting examples and publishable peers to ^0.0.0 and failing bun install. Name the private root @samva/integrations and ignore it in the release graph.
Rename the tegami-pr workflows to release-plan and release-plan-comment, and lowercase every workflow, job, and step name. Rename the preview artifact to release-plan and update the docs reference.
@greptile-apps

greptile-apps Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the previous lockfile inconsistency is fixed and no new actionable failures remain.

Fix All in CodexFindings

  1. P1 Root lockfile name is stale ▶

Summary

The PR repairs release drafting by giving the private repository root a distinct scoped package name and excluding it from Tegami’s release graph. It also standardizes GitHub Actions naming and aligns the release-plan producer, artifact, consumer, and documentation.

  • Renames the private root package to @samva/workspace.
  • Keeps the root lockfile metadata and Tegami ignore entry synchronized with that name.
  • Renames and normalizes release-plan workflows, jobs, steps, and artifacts.
  • Updates release documentation for the renamed workflows.
  • The previous stale-lockfile finding is fully fixed because package.json and bun.lock now both identify the root workspace as @samva/workspace.
Diagram
sequenceDiagram
    participant PR as Pull request
    participant Plan as release plan workflow
    participant Artifact as release-plan artifact
    participant Comment as release plan comment workflow
    participant GH as Pull request comments

    PR->>Plan: Trigger on pull_request
    Plan->>Plan: Generate release-plan.md
    Plan->>Artifact: Upload release-plan
    Artifact-->>Comment: Download after successful workflow_run
    Comment->>GH: Post generated release plan
Loading

Reviews (3) · Last reviewed commit: "chore(release): name the private workspa..."

Comment thread package.json Outdated
The root workspaces entry still carried the old package name. Update it to @samva/integrations; bun install and bun install --frozen-lockfile both report no further changes.
Use @samva/workspace for the private root package, mirroring @samva/platform in the platform repo, across package.json, the Tegami ignore list, and the lockfile root entry.
@aryasaatvik
aryasaatvik merged commit ee8f218 into main Sep 19, 2026
3 checks passed
@aryasaatvik
aryasaatvik deleted the fix/release-workflow-naming branch September 19, 2026 10:58
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