docs: added AspireC4 aspire hosting to the docs site, community area#2952
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds an AspireC4 community MDX docs page (setup, quick-start, data flow, state mapping, links, modes, exclusions, limitations), a DSL styling tip for light/dark SVG icons, a changeset for ChangesAspireC4 Community Documentation
🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers:
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Adds a new Community documentation page to the docs site for AspireC4, a .NET Aspire hosting/extension that generates live LikeC4 diagrams from the Aspire resource graph.
Changes:
- Added a new Community docs page describing AspireC4 setup, behavior, and configuration options.
- Added a changeset to publish the docs update as a patch to
@likec4/docs-astro.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| apps/docs/src/content/docs/tooling/Community/aspirec4.mdx | New Community documentation page for AspireC4 (overview, quick start, configuration, limitations). |
| .changeset/add-aspirec4-community-docs.md | Changeset to release the docs update as a patch. |
| label: 'AspireC4 - Aspire Integration' | ||
| --- | ||
|
|
||
| import { LinkCard, CardGrid, LinkButton } from '@astrojs/starlight/components'; |
🦋 Changeset detectedLatest commit: 91020dd The changes in this PR will be included in the next version bump. This PR includes changesets to release 22 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/src/content/docs/dsl/styling.mdx`:
- Around line 235-239: Fix the wording and the code-fence language in the tip:
change the phrase "dark mode's" to "dark modes" in the sentence that begins
"When using SVGs, you can bake the light and dark mode's into a single file",
and change the code fence language token from "```xmlm" to "```xml" (the fenced
code block immediately following that sentence).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c8b00cf5-481a-4ee9-a479-aed8ed591803
⛔ Files ignored due to path filters (3)
apps/docs/src/assets/logo.svgis excluded by!**/*.svgapps/docs/src/assets/tooling/community/aspirec4/makerstack-basic.pngis excluded by!**/*.pngapps/docs/src/assets/tooling/community/aspirec4/makerstack-enhanced.pngis excluded by!**/*.png
📒 Files selected for processing (2)
apps/docs/src/content/docs/dsl/styling.mdxapps/docs/src/content/docs/tooling/Community/aspirec4.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
- apps/docs/src/content/docs/tooling/Community/aspirec4.mdx
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: Kieron Lanning <kieronlanning@users.noreply.github.com>
davydkov
left a comment
There was a problem hiding this comment.
@kieronlanning Thank you!
Please fix failing build and merge it
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
130-139:⚠️ Potential issue | 🔴 Critical | ⚡ Quick winAdd
@openrouter/sdkto devDependencies — it's listed inonlyBuiltDependenciesbut not declared as a dependency.The
onlyBuiltDependenciesconfiguration controls which packages pnpm treats as build-only, but every package in that list must be declared independenciesordevDependencies. Currently,@openrouter/sdkappears only inonlyBuiltDependencies(line 131) and is missing from thedevDependenciessection. This will cause pnpm resolution failures.Add
@openrouter/sdktodevDependencieswith an appropriate version, or remove it fromonlyBuiltDependenciesif it's not needed.(Note:
@google/genaiandprotobufjsremovals are complete and verified.)🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@package.json` around lines 130 - 139, The package "`@openrouter/sdk`" is listed in onlyBuiltDependencies but not declared, causing pnpm resolution failures; update package.json by adding "`@openrouter/sdk`" to devDependencies with a suitable semver (or alternatively remove "`@openrouter/sdk`" from the onlyBuiltDependencies array if it shouldn't be build-only), ensuring you edit the devDependencies section and the onlyBuiltDependencies array entries so the package appears in both places (or is removed) and re-run install to verify resolution.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@package.json`:
- Around line 130-139: The package "`@openrouter/sdk`" is listed in
onlyBuiltDependencies but not declared, causing pnpm resolution failures; update
package.json by adding "`@openrouter/sdk`" to devDependencies with a suitable
semver (or alternatively remove "`@openrouter/sdk`" from the onlyBuiltDependencies
array if it shouldn't be build-only), ensuring you edit the devDependencies
section and the onlyBuiltDependencies array entries so the package appears in
both places (or is removed) and re-run install to verify resolution.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 7a028179-231e-4abf-87b5-da35e90a5265
📒 Files selected for processing (3)
.devcontainer/devcontainer-lock.jsonapps/docs/src/content/docs/dsl/styling.mdxpackage.json
✅ Files skipped from review due to trivial changes (2)
- apps/docs/src/content/docs/dsl/styling.mdx
- .devcontainer/devcontainer-lock.json
Interestinly, locally that showed as a warning - but it was so early in the build phase I'd missed it in the terminal... so it all looked good. Anyway, should be sorted now. |
Checklist
mainbefore creating this PR.pnpm typecheckandpnpm test./changeset-generatorSKILL).