Skip to content

Trigger a parent git repo prepublish when packages are rebuilt#19

Open
eoghanmurray wants to merge 1 commit intomasterfrom
parent-repo-prepublish
Open

Trigger a parent git repo prepublish when packages are rebuilt#19
eoghanmurray wants to merge 1 commit intomasterfrom
parent-repo-prepublish

Conversation

@eoghanmurray
Copy link
Copy Markdown
Collaborator

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 12, 2026

⚠️ No Changeset found

Latest commit: e6fb3a2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

"start": "vite",
"check-types": "svelte-check --tsconfig ./tsconfig.json",
"prepublish": "yarn build",
"prepublish": "yarn build && sh ../../../prepublish-rrweb.sh",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CRITICAL: The referenced script prepublish-rrweb.sh does not exist in the repository. This will cause the publish command to fail when running npm publish or yarn publish.

"retest:update": "cross-env PUPPETEER_HEADLESS=true yarn retest --update",
"check-types": "tsc -noEmit",
"prepublish": "tsc -noEmit && vite build",
"prepublish": "tsc -noEmit && vite build && sh ../../../prepublish-rrweb.sh",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CRITICAL: The referenced script prepublish-rrweb.sh does not exist in the repository. This will cause the publish command to fail when running npm publish or yarn publish.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented Mar 12, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 2
WARNING 0
SUGGESTION 0
Issue Details (click to expand)

CRITICAL

File Line Issue
packages/rrweb-player/package.json 32 Referenced script prepublish-rrweb.sh does not exist in repository
packages/ws-client/package.json 24 Referenced script prepublish-rrweb.sh does not exist in repository
Other Observations (not in diff)

No additional observations.

Files Reviewed (2 files)
  • packages/rrweb-player/package.json - 1 issue
  • packages/ws-client/package.json - 1 issue

Fix these issues in Kilo Cloud: https://app.kilo.ai/cloud-agent-fork/review/5f2187dd-f9ac-4f8d-9db6-9e713915902f

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