[seanspeaks]

Summary

Allows modelName, setAuthParams, and getAuthorizationRequirements in the api-module definition schema.

Also includes

Port of PR #639’s EventBridge scheduler lazy-load (@aws-sdk/client-scheduler loaded in the adapter constructor, declared as a core dependency) so CLI/canary consumers of @friggframework/core do not crash on require. #639 itself was left untouched (#639 is stale vs next); only the scheduler fix was ported here for the #701 → #702 stack.

  

📦 Published PR as canary version: 2.0.0--canary.701.32f702a.0

✨ Test out this PR locally via:

npm install @friggframework/admin-scripts@2.0.0--canary.701.32f702a.0
npm install @friggframework/core@2.0.0--canary.701.32f702a.0
npm install @friggframework/devtools@2.0.0--canary.701.32f702a.0
npm install @friggframework/eslint-config@2.0.0--canary.701.32f702a.0
npm install @friggframework/prettier-config@2.0.0--canary.701.32f702a.0
npm install @friggframework/schemas@2.0.0--canary.701.32f702a.0
npm install @friggframework/serverless-plugin@2.0.0--canary.701.32f702a.0
npm install @friggframework/test@2.0.0--canary.701.32f702a.0
npm install @friggframework/ui@2.0.0--canary.701.32f702a.0
# or 
yarn add @friggframework/admin-scripts@2.0.0--canary.701.32f702a.0
yarn add @friggframework/core@2.0.0--canary.701.32f702a.0
yarn add @friggframework/devtools@2.0.0--canary.701.32f702a.0
yarn add @friggframework/eslint-config@2.0.0--canary.701.32f702a.0
yarn add @friggframework/prettier-config@2.0.0--canary.701.32f702a.0
yarn add @friggframework/schemas@2.0.0--canary.701.32f702a.0
yarn add @friggframework/serverless-plugin@2.0.0--canary.701.32f702a.0
yarn add @friggframework/test@2.0.0--canary.701.32f702a.0
yarn add @friggframework/ui@2.0.0--canary.701.32f702a.0

---

Note

Low Risk
Changes are schema/CLI validation and documentation; they do not alter runtime auth or deployment behavior unless consumers relied on previously invalid module shapes.

^{Reviewed by Cursor Bugbot for commit 32f702a. Bugbot is set up for automated code reviews on this repo. Configure here.}

[netlify]

✅ Deploy Preview for friggframework-org canceled.

Name	Link
🔨 Latest commit	32f702a
🔍 Latest deploy log	https://app.netlify.com/projects/friggframework-org/deploys/6adc9d134eb861e21ee21dd5

[cursor]

Frigg 2.0.1 review (Group 2) — COMMENT only

Checklist (with #666 / #702)

• Allow modelName, setAuthParams, getAuthorizationRequirements on api-module definition schema: pass
• Consistent with #666 structural checks: pass (checkModuleDefinition still requires the auth method functions; schema now accepts the descriptors the view emits)

Verdict (would): would approve

Findings

None blocking. toApiModuleDefinitionSchemaView correctly turns functions into { type: 'function' } descriptors. Note: modelName remains optional in schema while v1 entity routing still needs a reliable name (modelName ?? moduleName) once getEntityModelFromDefinition is restored (#667 finding).

[cursor]

Follow-up (COMMENT)

should-fix (merge with #705): api-module-definition.schema.json still uses the legacy $id (https://schemas.friggframework.org/...). #705 rewrites the same file to https://friggframework.org/schemas/api-module-definition/v1.0.json.

Scenario: Merge order without reconciling $id + re-running sync:hosted → hosted v1.0 missing modelName / setAuthParams / getAuthorizationRequirements, or a conflict that drops one side’s changes. Keep the field additions and take #705’s $id, then sync.

[sonarqubecloud]

Quality Gate failed

Failed conditions
19.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[claude]

⚠️ Code review skipped — your organization has no extra usage available to pay for this review.

If your organization's extra usage balance is empty, an organization admin can add extra usage credits at claude.ai/admin-settings/usage. If its monthly spend limit was reached, an admin can raise it on the same page. If neither applies, contact Anthropic support.

Once extra usage is available, someone with write access to this repository can comment @claude review on this pull request to trigger a review.

[seanspeaks]

Rebase note for reviewers (rebased onto next after #666, head 32f702aa):

• feat(devtools): frigg validate --module and api-module schema fields #702 (frigg validate --module) is merged into this branch.
• Older copies of feat(devtools): frigg validate, and make the app-definition schema match the runtime (ADR-051) #666's commits were dropped in favour of the merged feat(devtools): frigg validate, and make the app-definition schema match the runtime (ADR-051) #666 (a25325a1), along with an empty "trigger canary" commit.
• The scheduler lazy-load port is dropped too, because it's already on next via fix: lazy-load scheduler AWS SDK in core and repair frigg init scaffolding #639. @aws-sdk/client-scheduler stays at next's ^3.588.0. The PR description above still mentions it; ignore that part.
• packages/schemas/index.js and package.json keep both feat(devtools): frigg validate, and make the app-definition schema match the runtime (ADR-051) #666's app-definition.js exports and this PR's api-module-definition.js exports.
• Frigg CI is green: https://github.com/friggframework/frigg/actions/runs/40091933711

🤖 Generated with Claude Code

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 32f702a. Configure here.}

[cursor]

Duplicate package description is ignored

Low Severity

The new description is overwritten by a later empty description in the same package.json. JSON parsers keep the last key, so npm publishes @friggframework/core with an empty description.

Additional Locations (1)

• packages/core/package.json#L91-L92

 

^{Reviewed by Cursor Bugbot for commit 32f702a. Configure here.}