Skip to content

docs(cli): add Nub app runner recipe - #1947

Closed
colinhacks wants to merge 1 commit into
aws:mainfrom
colinhacks:docs/cdk-nub-app-runner
Closed

docs(cli): add Nub app runner recipe#1947
colinhacks wants to merge 1 commit into
aws:mainfrom
colinhacks:docs/cdk-nub-app-runner

Conversation

@colinhacks

@colinhacks colinhacks commented Sep 5, 2026

Copy link
Copy Markdown

Adds an opt-in TypeScript application-runner recipe for Nub.

  • Pins @nubjs/nub to 0.8.3 and keeps the template type-check command.
  • Adds the TypeScript init integration test. It synthesizes with the default runner and Nub, then compares the resulting assembly.

Validation: a clean Linux VM ran the published @nubjs/nub@0.8.3 package through npx, type-checked both commands, synthesized with context, and compared all assembly files. Creation-stack metadata differs only in terminal runner frames; the test normalizes those frames and compares all remaining metadata.

The PR remains ready for review while the repository CI and init-template integration matrix run remotely.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Copilot AI lite review requested due to automatic review settings September 5, 2026 19:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@aws-cdk-automation
aws-cdk-automation requested a review from a team September 5, 2026 19:20
@github-actions github-actions Bot added the p2 label Sep 5, 2026
@colinhacks
colinhacks marked this pull request as draft September 5, 2026 19:21
auto-merge was automatically disabled September 5, 2026 19:21

Pull request was converted to draft

@colinhacks
colinhacks force-pushed the docs/cdk-nub-app-runner branch from a205323 to 14b89e3 Compare September 5, 2026 19:29
@colinhacks
colinhacks marked this pull request as ready for review September 5, 2026 19:30
@colinhacks
colinhacks force-pushed the docs/cdk-nub-app-runner branch from 14b89e3 to e66e7b4 Compare September 5, 2026 19:35
@colinhacks

Copy link
Copy Markdown
Author

The existing app command already supports this runner without a template change.

@colinhacks colinhacks closed this Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants