Closed
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
📝 WalkthroughWalkthroughVersion bump from 3.73.0 to 3.74.0 across the entire monorepo. All package.json files and CHANGELOG.md entries updated to reflect the new release date (2026-03-30) with dependency version ranges synchronized. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the project version to 3.74.0 across all packages and updates internal dependencies. It includes bug fixes and new features for the agent-runtime and agent-tracing modules. Feedback was provided to remove excessive blank lines in the updated changelog files to improve readability.
Comment on lines
+21
to
+24
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+13
to
+16
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+20
to
+23
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
Comment on lines
+10
to
+13
|
|
||
|
|
||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
These extra blank lines can be removed for better readability.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (6)
plugin/common/package.json (1)
1-49:⚠️ Potential issue | 🟠 MajorAdd
eggPluginandeggModulefields toplugin/common/package.json.Per coding guidelines, plugin packages must include
eggPluginwithnameanddependenciesproperties, andeggModulewithnameproperty for standalone mode support. All other plugins in the repository (ajv, aop, config, controller, dal, etc.) have these fields;plugin/common/package.jsonis the only one missing them.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/common/package.json` around lines 1 - 49, Add the missing top-level metadata fields to package.json: add an "eggPlugin" object with a "name" set to the package's plugin name (use the package "name" value or its short plugin name) and a "dependencies" object listing any plugin runtime deps, and add an "eggModule" object with a "name" property (same plugin/module name) to enable standalone mode; update the existing package.json JSON to include these two fields (eggPlugin and eggModule) alongside the current keys.plugin/tegg/package.json (1)
1-10:⚠️ Potential issue | 🟠 MajorAdd missing
eggModulemetadata for standalone support.
plugin/*/package.jsonrequires aneggModulefield with anameproperty, but this manifest currently only defineseggPlugin.As per coding guidelines: "Plugin `package.json` files must include `eggPlugin` field with `name` and `dependencies` properties, and `eggModule` field with `name` property for standalone mode support".🔧 Proposed fix
{ "name": "@eggjs/tegg-plugin", "eggPlugin": { "name": "tegg", "dependencies": [ "teggConfig" ] }, + "eggModule": { + "name": "tegg" + }, "version": "3.74.0",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/tegg/package.json` around lines 1 - 10, The package.json currently defines only the eggPlugin metadata (eggPlugin.name "tegg" and eggPlugin.dependencies ["teggConfig"]) but is missing the required eggModule metadata for standalone support; update the manifest to add an eggModule object with a name property (matching the module name, e.g., "tegg") so both eggPlugin and eggModule are present; ensure eggModule.name is a string and the overall JSON remains valid.plugin/eventbus/package.json (1)
4-10:⚠️ Potential issue | 🟠 MajorAdd missing
eggModulefield for standalone mode support.The
plugin/eventbus/package.jsonhas aneggPluginfield but is missing the requiredeggModulefield. As per coding guidelines, plugins supporting standalone mode require aneggModulefield inpackage.jsonwith anameproperty.🔧 Suggested fix
"eggPlugin": { "name": "eventbusModule", "strict": false, "dependencies": [ "tegg" ] }, + "eggModule": { + "name": "eventbusModule" + }, "description": "tegg event plugin",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/eventbus/package.json` around lines 4 - 10, The package.json currently defines an eggPlugin object but lacks the required eggModule entry for standalone mode; add an eggModule field (e.g., "eggModule": { "name": "eventbusModule" }) to package.json so the module name is declared for standalone support, ensuring the value matches the plugin name used in the existing "eggPlugin" block.plugin/orm/package.json (1)
3-8:⚠️ Potential issue | 🟠 MajorAdd missing
eggModulefield for standalone mode support.The
plugin/orm/package.jsonhas aneggPluginfield but is missing the requiredeggModulefield. As per coding guidelines, plugins supporting standalone mode require aneggModulefield inpackage.jsonwith anameproperty.🔧 Suggested fix
"eggPlugin": { "name": "teggOrm", "dependencies": [ "tegg" ] }, + "eggModule": { + "name": "teggOrm" + }, "version": "3.74.0",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/orm/package.json` around lines 3 - 8, Add the missing eggModule field to package.json to support standalone mode: under the existing eggPlugin block (which contains "name": "teggOrm"), add an "eggModule" object with a "name" property (for example the same "teggOrm" value) so package.json contains both "eggPlugin" and "eggModule" entries; update the package.json entry that currently lists only eggPlugin to include eggModule with a name to meet the standalone-mode requirement.plugin/controller/package.json (1)
1-10:⚠️ Potential issue | 🟠 MajorAdd required
eggModulefield in plugin manifest.This plugin manifest is missing
eggModule.name, which is required for standalone mode support/module scanning in this repo.As per coding guidelines: "Plugin `package.json` files must include `eggPlugin` field with `name` and `dependencies` properties, and `eggModule` field with `name` property for standalone mode support".Proposed fix
"eggPlugin": { "name": "teggController", "strict": false, "dependencies": [ "tegg" ] }, + "eggModule": { + "name": "teggController" + }, "version": "3.74.0",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/controller/package.json` around lines 1 - 10, The package manifest is missing the required eggModule.name; add an "eggModule" object with a "name" property (for example "tegg-controller-plugin" or the package name) alongside the existing "eggPlugin" object to support standalone mode/module scanning, and ensure the JSON stays valid (e.g., remove the trailing comma after "version" if present). Update the package.json to include eggModule.name and validate that "eggPlugin" (name: "teggController") and "eggModule.name" are present and correct.plugin/schedule/package.json (1)
1-10:⚠️ Potential issue | 🟠 MajorAdd required
eggModulemanifest field for standalone mode.
plugin/*/package.jsonrequireseggModule.name, but this manifest only defineseggPlugin. Please addeggModuleto keep standalone module scanning support intact.As per coding guidelines: "Plugin `package.json` files must include `eggPlugin` field with `name` and `dependencies` properties, and `eggModule` field with `name` property for standalone mode support".Proposed fix
"eggPlugin": { "name": "teggSchedule", "dependencies": [ "tegg", "schedule" ] }, + "eggModule": { + "name": "teggSchedule" + }, "version": "3.74.0",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugin/schedule/package.json` around lines 1 - 10, Add an eggModule manifest to package.json so standalone scanning works: insert an "eggModule" object with a "name" property (e.g. set eggModule.name to the same identifier used in "eggPlugin.name", "teggSchedule") alongside the existing "eggPlugin" entry; ensure the new eggModule field is a top-level key in package.json and contains only the required "name" string property.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@core/agent-tracing/CHANGELOG.md`:
- Around line 9-18: Update the markdown headings in
core/agent-tracing/CHANGELOG.md by changing the two subsection headings "### Bug
Fixes" and "### Features" to level-2 headings ("## Bug Fixes" and "## Features")
so they correctly follow the top-level H1; edit the lines containing those exact
heading texts and ensure no other heading syntax or content is altered.
In `@core/standalone-decorator/package.json`:
- Around line 38-40: The package.json for the core/standalone-decorator package
is missing required dependencies; add entries for "@eggjs/core-decorator" and
"@eggjs/tegg-types" under the "dependencies" object in package.json so the
decorator package follows repository rules; pick appropriate semver ranges
consistent with other core/*-decorator packages in the repo (or align with the
repo's current versions) and ensure package.json remains valid JSON.
In `@plugin/config/package.json`:
- Around line 3-6: The plugin manifest is missing required fields: update
package.json to add eggPlugin.dependencies (map any runtime plugin dependencies)
under the existing eggPlugin object and add an eggModule object with a name
property for standalone mode support; keep the existing eggPlugin.name value
"teggConfig" and populate eggPlugin.dependencies with the plugin keys this
module depends on (or an empty object if none) and set eggModule.name to the
module identifier used for module loading so dependency wiring and standalone
mode work correctly.
In `@plugin/mcp-proxy/package.json`:
- Around line 4-6: The package.json currently defines an "eggPlugin" object
(name "mcpProxy") but is missing the required "eggModule" field for standalone
mode; add an "eggModule" entry with a "name" property (e.g., "mcpProxy")
alongside "eggPlugin" so both "eggPlugin" and "eggModule" are present and the
module supports standalone mode.
---
Outside diff comments:
In `@plugin/common/package.json`:
- Around line 1-49: Add the missing top-level metadata fields to package.json:
add an "eggPlugin" object with a "name" set to the package's plugin name (use
the package "name" value or its short plugin name) and a "dependencies" object
listing any plugin runtime deps, and add an "eggModule" object with a "name"
property (same plugin/module name) to enable standalone mode; update the
existing package.json JSON to include these two fields (eggPlugin and eggModule)
alongside the current keys.
In `@plugin/controller/package.json`:
- Around line 1-10: The package manifest is missing the required eggModule.name;
add an "eggModule" object with a "name" property (for example
"tegg-controller-plugin" or the package name) alongside the existing "eggPlugin"
object to support standalone mode/module scanning, and ensure the JSON stays
valid (e.g., remove the trailing comma after "version" if present). Update the
package.json to include eggModule.name and validate that "eggPlugin" (name:
"teggController") and "eggModule.name" are present and correct.
In `@plugin/eventbus/package.json`:
- Around line 4-10: The package.json currently defines an eggPlugin object but
lacks the required eggModule entry for standalone mode; add an eggModule field
(e.g., "eggModule": { "name": "eventbusModule" }) to package.json so the module
name is declared for standalone support, ensuring the value matches the plugin
name used in the existing "eggPlugin" block.
In `@plugin/orm/package.json`:
- Around line 3-8: Add the missing eggModule field to package.json to support
standalone mode: under the existing eggPlugin block (which contains "name":
"teggOrm"), add an "eggModule" object with a "name" property (for example the
same "teggOrm" value) so package.json contains both "eggPlugin" and "eggModule"
entries; update the package.json entry that currently lists only eggPlugin to
include eggModule with a name to meet the standalone-mode requirement.
In `@plugin/schedule/package.json`:
- Around line 1-10: Add an eggModule manifest to package.json so standalone
scanning works: insert an "eggModule" object with a "name" property (e.g. set
eggModule.name to the same identifier used in "eggPlugin.name", "teggSchedule")
alongside the existing "eggPlugin" entry; ensure the new eggModule field is a
top-level key in package.json and contains only the required "name" string
property.
In `@plugin/tegg/package.json`:
- Around line 1-10: The package.json currently defines only the eggPlugin
metadata (eggPlugin.name "tegg" and eggPlugin.dependencies ["teggConfig"]) but
is missing the required eggModule metadata for standalone support; update the
manifest to add an eggModule object with a name property (matching the module
name, e.g., "tegg") so both eggPlugin and eggModule are present; ensure
eggModule.name is a string and the overall JSON remains valid.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 701e8e8d-ecd9-4615-bdcd-08dd89b2f1ea
📒 Files selected for processing (90)
CHANGELOG.mdcore/agent-runtime/CHANGELOG.mdcore/agent-runtime/package.jsoncore/agent-tracing/CHANGELOG.mdcore/agent-tracing/package.jsoncore/ajv-decorator/CHANGELOG.mdcore/ajv-decorator/package.jsoncore/aop-decorator/CHANGELOG.mdcore/aop-decorator/package.jsoncore/aop-runtime/CHANGELOG.mdcore/aop-runtime/package.jsoncore/background-task/CHANGELOG.mdcore/background-task/package.jsoncore/common-util/CHANGELOG.mdcore/common-util/package.jsoncore/controller-decorator/CHANGELOG.mdcore/controller-decorator/package.jsoncore/core-decorator/CHANGELOG.mdcore/core-decorator/package.jsoncore/dal-decorator/CHANGELOG.mdcore/dal-decorator/package.jsoncore/dal-runtime/CHANGELOG.mdcore/dal-runtime/package.jsoncore/dynamic-inject-runtime/CHANGELOG.mdcore/dynamic-inject-runtime/package.jsoncore/dynamic-inject/CHANGELOG.mdcore/dynamic-inject/package.jsoncore/eventbus-decorator/CHANGELOG.mdcore/eventbus-decorator/package.jsoncore/eventbus-runtime/CHANGELOG.mdcore/eventbus-runtime/package.jsoncore/langchain-decorator/CHANGELOG.mdcore/langchain-decorator/package.jsoncore/lifecycle/CHANGELOG.mdcore/lifecycle/package.jsoncore/loader/CHANGELOG.mdcore/loader/package.jsoncore/mcp-client/CHANGELOG.mdcore/mcp-client/package.jsoncore/metadata/CHANGELOG.mdcore/metadata/package.jsoncore/orm-decorator/CHANGELOG.mdcore/orm-decorator/package.jsoncore/runtime/CHANGELOG.mdcore/runtime/package.jsoncore/schedule-decorator/CHANGELOG.mdcore/schedule-decorator/package.jsoncore/standalone-decorator/CHANGELOG.mdcore/standalone-decorator/package.jsoncore/tegg/CHANGELOG.mdcore/tegg/package.jsoncore/test-util/CHANGELOG.mdcore/test-util/package.jsoncore/transaction-decorator/CHANGELOG.mdcore/transaction-decorator/package.jsoncore/types/CHANGELOG.mdcore/types/package.jsoncore/vitest/CHANGELOG.mdcore/vitest/package.jsonlerna.jsonplugin/ajv/CHANGELOG.mdplugin/ajv/package.jsonplugin/aop/CHANGELOG.mdplugin/aop/package.jsonplugin/common/CHANGELOG.mdplugin/common/package.jsonplugin/config/CHANGELOG.mdplugin/config/package.jsonplugin/controller/CHANGELOG.mdplugin/controller/package.jsonplugin/dal/CHANGELOG.mdplugin/dal/package.jsonplugin/dns-cache/CHANGELOG.mdplugin/dns-cache/package.jsonplugin/eventbus/CHANGELOG.mdplugin/eventbus/package.jsonplugin/langchain/CHANGELOG.mdplugin/langchain/package.jsonplugin/mcp-client/CHANGELOG.mdplugin/mcp-client/package.jsonplugin/mcp-proxy/CHANGELOG.mdplugin/mcp-proxy/package.jsonplugin/orm/CHANGELOG.mdplugin/orm/package.jsonplugin/schedule/CHANGELOG.mdplugin/schedule/package.jsonplugin/tegg/CHANGELOG.mdplugin/tegg/package.jsonstandalone/standalone/CHANGELOG.mdstandalone/standalone/package.json
Comment on lines
+9
to
+18
| ### Bug Fixes | ||
|
|
||
| * **agent-tracing:** add eggModule declaration for tegg module scanning ([#416](https://github.com/eggjs/tegg/issues/416)) ([d169cab](https://github.com/eggjs/tegg/commit/d169cabeb0580b96c320ac6c067d3d9828639a32)) | ||
| * **agent-tracing:** separate traceId and sessionId in createSession ([#417](https://github.com/eggjs/tegg/issues/417)) ([c760776](https://github.com/eggjs/tegg/commit/c7607761d1e85cbf91cb74878e90af898672db3a)) | ||
|
|
||
|
|
||
| ### Features | ||
|
|
||
| * **agent-tracing:** collect assistant messages into root run outputs ([#421](https://github.com/eggjs/tegg/issues/421)) ([8dcebaf](https://github.com/eggjs/tegg/commit/8dcebafa19061d4a91161272409a2ac729d78341)) | ||
| * **agent-tracing:** rename TraceSession to Trace and support inputs in createTrace ([#420](https://github.com/eggjs/tegg/issues/420)) ([5471bda](https://github.com/eggjs/tegg/commit/5471bda05bd21ed5c60d82182a2807d09f9be097)) |
There was a problem hiding this comment.
Fix markdown heading levels.
The subsection headings (### Bug Fixes and ### Features) should use h2 (##) instead of h3 (###) to follow proper markdown hierarchy. After an h1 heading (#), the next level should be h2, not h3.
📝 Proposed fix for heading levels
-### Bug Fixes
+## Bug Fixes
* **agent-tracing:** add eggModule declaration for tegg module scanning ([`#416`](https://github.com/eggjs/tegg/issues/416)) ([d169cab](https://github.com/eggjs/tegg/commit/d169cabeb0580b96c320ac6c067d3d9828639a32))
* **agent-tracing:** separate traceId and sessionId in createSession ([`#417`](https://github.com/eggjs/tegg/issues/417)) ([c760776](https://github.com/eggjs/tegg/commit/c7607761d1e85cbf91cb74878e90af898672db3a))
-### Features
+## Features
* **agent-tracing:** collect assistant messages into root run outputs ([`#421`](https://github.com/eggjs/tegg/issues/421)) ([8dcebaf](https://github.com/eggjs/tegg/commit/8dcebafa19061d4a91161272409a2ac729d78341))
* **agent-tracing:** rename TraceSession to Trace and support inputs in createTrace ([`#420`](https://github.com/eggjs/tegg/issues/420)) ([5471bda](https://github.com/eggjs/tegg/commit/5471bda05bd21ed5c60d82182a2807d09f9be097))📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| ### Bug Fixes | |
| * **agent-tracing:** add eggModule declaration for tegg module scanning ([#416](https://github.com/eggjs/tegg/issues/416)) ([d169cab](https://github.com/eggjs/tegg/commit/d169cabeb0580b96c320ac6c067d3d9828639a32)) | |
| * **agent-tracing:** separate traceId and sessionId in createSession ([#417](https://github.com/eggjs/tegg/issues/417)) ([c760776](https://github.com/eggjs/tegg/commit/c7607761d1e85cbf91cb74878e90af898672db3a)) | |
| ### Features | |
| * **agent-tracing:** collect assistant messages into root run outputs ([#421](https://github.com/eggjs/tegg/issues/421)) ([8dcebaf](https://github.com/eggjs/tegg/commit/8dcebafa19061d4a91161272409a2ac729d78341)) | |
| * **agent-tracing:** rename TraceSession to Trace and support inputs in createTrace ([#420](https://github.com/eggjs/tegg/issues/420)) ([5471bda](https://github.com/eggjs/tegg/commit/5471bda05bd21ed5c60d82182a2807d09f9be097)) | |
| ## Bug Fixes | |
| * **agent-tracing:** add eggModule declaration for tegg module scanning ([`#416`](https://github.com/eggjs/tegg/issues/416)) ([d169cab](https://github.com/eggjs/tegg/commit/d169cabeb0580b96c320ac6c067d3d9828639a32)) | |
| * **agent-tracing:** separate traceId and sessionId in createSession ([`#417`](https://github.com/eggjs/tegg/issues/417)) ([c760776](https://github.com/eggjs/tegg/commit/c7607761d1e85cbf91cb74878e90af898672db3a)) | |
| ## Features | |
| * **agent-tracing:** collect assistant messages into root run outputs ([`#421`](https://github.com/eggjs/tegg/issues/421)) ([8dcebaf](https://github.com/eggjs/tegg/commit/8dcebafa19061d4a91161272409a2ac729d78341)) | |
| * **agent-tracing:** rename TraceSession to Trace and support inputs in createTrace ([`#420`](https://github.com/eggjs/tegg/issues/420)) ([5471bda](https://github.com/eggjs/tegg/commit/5471bda05bd21ed5c60d82182a2807d09f9be097)) |
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 9-9: Heading levels should only increment by one level at a time
Expected: h2; Actual: h3
(MD001, heading-increment)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@core/agent-tracing/CHANGELOG.md` around lines 9 - 18, Update the markdown
headings in core/agent-tracing/CHANGELOG.md by changing the two subsection
headings "### Bug Fixes" and "### Features" to level-2 headings ("## Bug Fixes"
and "## Features") so they correctly follow the top-level H1; edit the lines
containing those exact heading texts and ensure no other heading syntax or
content is altered.
Comment on lines
38
to
40
| "dependencies": { | ||
| "@eggjs/tegg-common-util": "^3.73.0", | ||
| "@eggjs/tegg-common-util": "^3.74.0", | ||
| "reflect-metadata": "^0.1.13" |
There was a problem hiding this comment.
Decorator package is missing required @eggjs/core-decorator and @eggjs/tegg-types dependencies.
Please add both required dependencies to align with repository packaging rules for core/*-decorator packages.
Suggested dependency patch
"dependencies": {
+ "@eggjs/core-decorator": "^3.74.0",
"@eggjs/tegg-common-util": "^3.74.0",
+ "@eggjs/tegg-types": "^3.74.0",
"reflect-metadata": "^0.1.13"
},As per coding guidelines, "Decorator package package.json must include dependencies on @eggjs/core-decorator and @eggjs/tegg-types".
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "dependencies": { | |
| "@eggjs/tegg-common-util": "^3.73.0", | |
| "@eggjs/tegg-common-util": "^3.74.0", | |
| "reflect-metadata": "^0.1.13" | |
| "dependencies": { | |
| "@eggjs/core-decorator": "^3.74.0", | |
| "@eggjs/tegg-common-util": "^3.74.0", | |
| "@eggjs/tegg-types": "^3.74.0", | |
| "reflect-metadata": "^0.1.13" |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@core/standalone-decorator/package.json` around lines 38 - 40, The
package.json for the core/standalone-decorator package is missing required
dependencies; add entries for "@eggjs/core-decorator" and "@eggjs/tegg-types"
under the "dependencies" object in package.json so the decorator package follows
repository rules; pick appropriate semver ranges consistent with other
core/*-decorator packages in the repo (or align with the repo's current
versions) and ensure package.json remains valid JSON.
Comment on lines
3
to
+6
| "eggPlugin": { | ||
| "name": "teggConfig" | ||
| }, | ||
| "version": "3.73.0", | ||
| "version": "3.74.0", |
There was a problem hiding this comment.
Add required eggPlugin.dependencies and eggModule.name fields to plugin manifest.
plugin/config/package.json still lacks required metadata for plugin dependency wiring and standalone mode support.
Suggested manifest patch
{
"name": "@eggjs/tegg-config",
"eggPlugin": {
- "name": "teggConfig"
+ "name": "teggConfig",
+ "dependencies": []
},
+ "eggModule": {
+ "name": "teggConfig"
+ },
"version": "3.74.0",
...
}As per coding guidelines, "Plugin package.json files must include eggPlugin field with name and dependencies properties, and eggModule field with name property for standalone mode support".
Also applies to: 46-46
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@plugin/config/package.json` around lines 3 - 6, The plugin manifest is
missing required fields: update package.json to add eggPlugin.dependencies (map
any runtime plugin dependencies) under the existing eggPlugin object and add an
eggModule object with a name property for standalone mode support; keep the
existing eggPlugin.name value "teggConfig" and populate eggPlugin.dependencies
with the plugin keys this module depends on (or an empty object if none) and set
eggModule.name to the module identifier used for module loading so dependency
wiring and standalone mode work correctly.
Comment on lines
4
to
6
| "eggPlugin": { | ||
| "name": "mcpProxy" | ||
| }, |
There was a problem hiding this comment.
Missing eggModule field for standalone mode support.
The plugin includes an eggPlugin field but is missing the required eggModule field. As per coding guidelines, plugins supporting standalone mode require an eggModule field with a name property.
📦 Proposed fix to add eggModule field
"eggPlugin": {
"name": "mcpProxy"
},
+ "eggModule": {
+ "name": "mcpProxy"
+ },
"types": "typings/index.d.ts",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@plugin/mcp-proxy/package.json` around lines 4 - 6, The package.json currently
defines an "eggPlugin" object (name "mcpProxy") but is missing the required
"eggModule" field for standalone mode; add an "eggModule" entry with a "name"
property (e.g., "mcpProxy") alongside "eggPlugin" so both "eggPlugin" and
"eggModule" are present and the module supports standalone mode.
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
1 participant
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.
Summary
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes – v3.74.0
Bug Fixes
Features
TraceAPI enhanced with new session identifier separation and input support in trace creationChores