chore(scripts): expose missing workspace test and typecheck aliases at the root - #266
Merged
karagozemin merged 2 commits intoAug 31, 2026
Merged
Conversation
…t the root Closes Sub-Rosa-Issue#247 Siete alias que faltaban, todos delegando al script del workspace con el mismo `pnpm --filter` que ya usa el resto del manifiesto: tlock:typecheck, keeper:typecheck, drand:test, drand:typecheck, receipt:test, receipt:typecheck, template:test Se agrego un alias solo donde el workspace ya definia el script y el root no lo exponia. Los siete corren y pasan. Sin cambios en ningun package.json de workspace ni en codigo fuente. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #247
Adds the seven root aliases that were missing. Each one delegates to the workspace script with the same
pnpm --filterform the rest of the manifest already uses.What was missing
I walked every workspace manifest and compared its
test/typecheckscripts against the root. An alias was added only where the workspace already defines the script and the root did not expose it:@sub-rosa/tlocktlock:typecheck@sub-rosa/keeperkeeper:typecheck@sub-rosa/drand-toolsdrand:test,drand:typecheck@sub-rosa/receipt-clireceipt:test,receipt:typecheck@sub-rosa/auction-templatetemplate:testThat matches the workspaces named in the issue.
tlock:testandkeeper:testalready existed, which is exactly the inconsistency described: some workspaces had one half of the pair and not the other.Naming
template:is the prefix the root already uses forauction-template(template:fixture,template:testnet,template:typecheck), sotemplate:testfollows it.drand:andreceipt:are new but keep the same<short>:<script>shape as every other entry.Each alias was run
No alias resolves to a script that does not exist.
Scope
Diff is the root manifest only. No workspace
package.jsonand no source file was touched, and no existing root command changed. New entries are appended rather than sorted in, to keep the diff readable.🤖 Generated with Claude Code