-
Notifications
You must be signed in to change notification settings - Fork 87
chore: Publish release/20250331-72e07e #169
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThis pull request removes multiple JSON files that were used to track minor changes, bug fixes, and feature additions for the Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant WS as Workflow Service
participant HT as History Tracker
U->>WS: Trigger asynchronous workflow
WS->>HT: Log workflow start
WS-->>WS: Process tasks asynchronously
WS->>HT: Record workflow completion
WS-->>U: Return workflow result
Possibly related PRs
Suggested reviewers
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (5)
packages/realtime-api/package.json (1)
24-24: Verify "type" Field ConsistencyThe changelog indicates that the
"type": "module"declaration should be removed from package.json files. However, it remains here. Please verify if its retention is intentional or if it should be removed for consistency.packages/realtime-api/CHANGELOG.md (1)
3-18: Changelog Entry for Version 1.1.1 ValidatedThe new changelog section clearly documents both new features (e.g., mobile support, network error handling, screen sharing support) and bug fixes (including the removal of
"type": "module"from package.json files). Please consider reviewing the repeated unit test mentions to ensure clarity.🧰 Tools
🪛 LanguageTool
[uncategorized] ~15-~15: This word is normally spelled with a hyphen.
Context: ... code - add unit test ### Bug Fixes - Hard code test_app_id for Debug - add unit test -...(HARD_CODE_COMPOUND)
packages/coze-js/CHANGELOG.md (1)
16-27: Review Bug Fixes Section for RedundanciesIn the Bug Fixes section, multiple entries mention "add unit test." It might help clarity to consolidate these entries or clarify their differences so that the changelog remains concise.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~18-~18: This word is normally spelled with a hyphen.
Context: ... and history tracking ### Bug Fixes - Hard code test_app_id for Debug - CR bug - add un...(HARD_CODE_COMPOUND)
[grammar] ~24-~24: You’ve repeated a verb. Did you mean to only write one of them?
Context: ...- Remove type: module from package.json files - file path error - fix speech bug - Publish f...(REPEATED_VERBS)
packages/realtime-api/CHANGELOG.json (1)
4-37: Structured JSON Changelog EntryThe new JSON entry for version
"1.1.1"accurately categorizes changes under"minor"and"patch". Note that "add unit test" appears in both sections; please verify if this redundancy is intentional.packages/coze-js/CHANGELOG.json (1)
4-67: New Changelog Entry for Version "1.1.1":
The new entry is well-structured with separate arrays for "minor" and "patch" comments. Verify that the"date"and"tag"fields align with your release process.Consistency & Clarity Improvements:
- In the "minor" section, the entries
"update README.md"(line 14) and"Update readme"(line 29) use inconsistent casing. It’s recommended to standardize these (e.g., always use"Update README.md").- Some comment texts (e.g.,
"CR bug"on line 40 and"ws-tools rslib config"on line 26) are terse. Consider expanding these to provide clearer context for future readers of the changelog.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (43)
common/changes/@coze/api/feat-api_2025-02-27-11-10.json(0 hunks)common/changes/@coze/api/feat-api_2025-02-27-12-38.json(0 hunks)common/changes/@coze/api/feat-api_2025-02-28-03-27.json(0 hunks)common/changes/@coze/api/feat-network_2025-03-05-07-11.json(0 hunks)common/changes/@coze/api/feat-network_2025-03-05-11-19.json(0 hunks)common/changes/@coze/api/feat-share_screen_2025-02-21-09-29.json(0 hunks)common/changes/@coze/api/feat-share_screen_2025-02-24-06-38.json(0 hunks)common/changes/@coze/api/feat-share_screen_2025-02-27-13-12.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-17-07-31.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-17-08-59.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-17-09-05.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-20-02-54.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-20-03-16.json(0 hunks)common/changes/@coze/api/feat-websocket-api_2025-02-20-05-13.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-07-10-41.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-09-03-21.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-10-05-12.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-10-06-06.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-11-06-32.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-11-06-43.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-28-07-06.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-28-07-07.json(0 hunks)common/changes/@coze/api/fix-bug_2025-03-30-09-25.json(0 hunks)common/changes/@coze/api/fix-speech_2025-03-12-13-51.json(0 hunks)common/changes/@coze/api/fix-speech_2025-03-14-03-48.json(0 hunks)common/changes/@coze/api/fix-speech_2025-03-14-04-06.json(0 hunks)common/changes/@coze/api/fix-speech_2025-03-14-04-11.json(0 hunks)common/changes/@coze/api/fix-workflow_2025-02-24-12-00.json(0 hunks)common/changes/@coze/realtime-api/feat-network_2025-03-04-05-18.json(0 hunks)common/changes/@coze/realtime-api/feat-network_2025-03-05-07-10.json(0 hunks)common/changes/@coze/realtime-api/feat-network_2025-03-05-12-16.json(0 hunks)common/changes/@coze/realtime-api/feat-realtime-api_2025-02-14-10-11.json(0 hunks)common/changes/@coze/realtime-api/feat-share_screen_2025-02-21-09-29.json(0 hunks)common/changes/@coze/realtime-api/feat-share_screen_2025-02-24-06-38.json(0 hunks)common/changes/@coze/realtime-api/feat-share_screen_2025-02-27-12-50.json(0 hunks)common/changes/@coze/realtime-api/feat-share_screen_2025-02-27-13-27.json(0 hunks)common/changes/@coze/realtime-api/fix-bug_2025-03-28-07-06.json(0 hunks)packages/coze-js/CHANGELOG.json(1 hunks)packages/coze-js/CHANGELOG.md(1 hunks)packages/coze-js/package.json(1 hunks)packages/realtime-api/CHANGELOG.json(1 hunks)packages/realtime-api/CHANGELOG.md(1 hunks)packages/realtime-api/package.json(1 hunks)
💤 Files with no reviewable changes (37)
- common/changes/@coze/api/fix-speech_2025-03-12-13-51.json
- common/changes/@coze/api/feat-network_2025-03-05-11-19.json
- common/changes/@coze/api/fix-bug_2025-03-10-06-06.json
- common/changes/@coze/api/fix-bug_2025-03-10-05-12.json
- common/changes/@coze/api/feat-share_screen_2025-02-21-09-29.json
- common/changes/@coze/api/fix-bug_2025-03-11-06-32.json
- common/changes/@coze/api/fix-speech_2025-03-14-04-11.json
- common/changes/@coze/api/fix-workflow_2025-02-24-12-00.json
- common/changes/@coze/realtime-api/feat-network_2025-03-04-05-18.json
- common/changes/@coze/api/feat-websocket-api_2025-02-20-02-54.json
- common/changes/@coze/api/feat-network_2025-03-05-07-11.json
- common/changes/@coze/api/fix-bug_2025-03-07-10-41.json
- common/changes/@coze/realtime-api/feat-share_screen_2025-02-24-06-38.json
- common/changes/@coze/api/feat-websocket-api_2025-02-17-09-05.json
- common/changes/@coze/realtime-api/fix-bug_2025-03-28-07-06.json
- common/changes/@coze/api/feat-share_screen_2025-02-24-06-38.json
- common/changes/@coze/api/fix-bug_2025-03-28-07-06.json
- common/changes/@coze/api/fix-speech_2025-03-14-04-06.json
- common/changes/@coze/api/feat-api_2025-02-28-03-27.json
- common/changes/@coze/api/fix-speech_2025-03-14-03-48.json
- common/changes/@coze/api/feat-api_2025-02-27-12-38.json
- common/changes/@coze/api/fix-bug_2025-03-09-03-21.json
- common/changes/@coze/api/feat-share_screen_2025-02-27-13-12.json
- common/changes/@coze/api/feat-api_2025-02-27-11-10.json
- common/changes/@coze/realtime-api/feat-share_screen_2025-02-27-12-50.json
- common/changes/@coze/realtime-api/feat-network_2025-03-05-07-10.json
- common/changes/@coze/api/feat-websocket-api_2025-02-20-03-16.json
- common/changes/@coze/api/fix-bug_2025-03-30-09-25.json
- common/changes/@coze/realtime-api/feat-share_screen_2025-02-21-09-29.json
- common/changes/@coze/realtime-api/feat-share_screen_2025-02-27-13-27.json
- common/changes/@coze/api/fix-bug_2025-03-11-06-43.json
- common/changes/@coze/api/fix-bug_2025-03-28-07-07.json
- common/changes/@coze/realtime-api/feat-network_2025-03-05-12-16.json
- common/changes/@coze/api/feat-websocket-api_2025-02-20-05-13.json
- common/changes/@coze/realtime-api/feat-realtime-api_2025-02-14-10-11.json
- common/changes/@coze/api/feat-websocket-api_2025-02-17-07-31.json
- common/changes/@coze/api/feat-websocket-api_2025-02-17-08-59.json
🧰 Additional context used
🪛 LanguageTool
packages/realtime-api/CHANGELOG.md
[uncategorized] ~15-~15: This word is normally spelled with a hyphen.
Context: ... code - add unit test ### Bug Fixes - Hard code test_app_id for Debug - add unit test -...
(HARD_CODE_COMPOUND)
packages/coze-js/CHANGELOG.md
[uncategorized] ~18-~18: This word is normally spelled with a hyphen.
Context: ... and history tracking ### Bug Fixes - Hard code test_app_id for Debug - CR bug - add un...
(HARD_CODE_COMPOUND)
[grammar] ~24-~24: You’ve repeated a verb. Did you mean to only write one of them?
Context: ...- Remove type: module from package.json files - file path error - fix speech bug - Publish f...
(REPEATED_VERBS)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Node.js v20 (ubuntu-latest)
🔇 Additional comments (3)
packages/coze-js/package.json (1)
3-3: Version Bump ConfirmationThe version update from
"1.1.0-beta.7"to"1.1.1"correctly reflects the transition from beta to stable. Ensure that any downstream dependencies or documentation referring to the version are updated accordingly.packages/realtime-api/package.json (1)
3-3: Version Update ConfirmedThe version bump from
"1.1.0-beta.2"to"1.1.1"aligns with the stable release strategy.packages/coze-js/CHANGELOG.md (1)
3-14: Comprehensive New Features DocumentationThe entry for version 1.1.1 is detailed and well-structured with the listing of new features such as playback timing, screen sharing, and async workflow execution.
Codecov ReportAll modified and coverable lines are covered by tests ✅ ❌ Your project status has failed because the head coverage (0.00%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage. @@ Coverage Diff @@
## main #169 +/- ##
=========================================
- Coverage 91.82% 0 -91.83%
=========================================
Files 117 0 -117
Lines 5469 0 -5469
Branches 1106 0 -1106
=========================================
- Hits 5022 0 -5022
+ Misses 447 0 -447
see 117 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Summary by CodeRabbit
New Features
Bug Fixes
Documentation