feat: add capabilities.json (platform-registry Phase D) - #16
Conversation
Publishes this repo to wave-foundation/frameworks/platform-registry. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
|
Warning Review limit reached
More reviews will be available in 31 minutes and 41 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
| { | ||
| "schemaVersion": "1", | ||
| "repo": "wave-av/sdk", | ||
| "version": "2.0.14", |
There was a problem hiding this comment.
Bug: The version in capabilities.json ("2.0.14") is inconsistent with the actual package version in package.json ("2.0.4"). This will publish incorrect metadata to the platform registry.
Severity: HIGH
Suggested Fix
Update the version field in capabilities.json to match the version in package.json. The value should be changed from "2.0.14" to "2.0.4". Consider adding a CI check to validate version consistency between these files in the future.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: capabilities.json#L4
Potential issue: The `version` field in `capabilities.json` is set to `"2.0.14"`, but
the actual package version defined in `package.json` is `"2.0.4"`. This metadata file is
consumed by the `wave-foundation/frameworks/platform-registry`. When this change is
merged, the registry will index a non-existent version of the SDK. This will cause
failures for any downstream systems or developers that rely on the registry for
dependency resolution or compatibility tracking, as they will be pointed to a version
that has not been published. Existing CI/CD pipelines do not validate the consistency
between these two files, so there are no safeguards to prevent this incorrect metadata
from being published.
Did we get this right? 👍 / 👎 to inform future reviews.
There was a problem hiding this comment.
1 issue found across 1 file
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
Bot review caught capabilities.json version drift from package.json. Aligned. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Backfills this repo into
wave-foundation/frameworks/platform-registry.🤖 Generated with Claude Code
Summary by cubic
Adds
capabilities.jsonto publishwave-av/sdkinto the platform registry (Phase D) for discovery, dependency mapping, and lifecycle tracking.New Features
schemaVersion: "1"andfoundationPin: "v1".repo: "wave-av/sdk",version: "2.0.4",lifecycle: "beta", maintainers[{"team":"sdk"}].wave-av/api-specandframeworks/platform-registry.sdk,public,typescript,@wave-av/sdk.Bug Fixes
versionto matchpackage.json(2.0.4).Written for commit 50a14bc. Summary will update on new commits.