Skip to content

feat:增加角色卡配置 override & 多语言 - #213

Merged
MagicalAstrogy merged 9 commits into
betafrom
mag/card_override
Aug 8, 2026
Merged

feat:增加角色卡配置 override & 多语言#213
MagicalAstrogy merged 9 commits into
betafrom
mag/card_override

Conversation

@MagicalAstrogy

Copy link
Copy Markdown
Owner

rt

@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

🔍 PR Test Report

🧪 Test Results

✅ All tests passed!

Type Passed Failed Total
Test Suites 24 0 24
Tests 480 0 480

📊 Coverage Report

Metric Coverage Details
Lines 67.1% 1514/2256
Statements 66.94% 1551/2317
Functions 71.63% 202/282
Branches 63.42% 725/1143

Coverage Thresholds

  • ✅ Lines: 67.1% (Passed, threshold: 50%)
  • ✅ Statements: 66.94% (Passed, threshold: 50%)
  • ✅ Functions: 71.63% (Passed, threshold: 50%)
  • ✅ Branches: 63.42% (Passed, threshold: 50%)

Generated at 2026-08-08T03:59:57.853Z

@MagicalAstrogy

Copy link
Copy Markdown
Owner Author

慢慢看...

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 40612ab147

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/main.ts
@MagicalAstrogy

Copy link
Copy Markdown
Owner Author

LGFM
周末就合吧

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 185c03875e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/function/character_override/index.ts
Comment thread src/main.ts
@MagicalAstrogy

Copy link
Copy Markdown
Owner Author

LGFM

@MagicalAstrogy
MagicalAstrogy merged commit 7009932 into beta Aug 8, 2026
1 check passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 629ab8c3b4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +584 to +589
if (target_entry) {
entry_uid = target_entry.uid;
Object.assign(target_entry, {
name: CHARACTER_SETTINGS_OVERRIDE_ENTRY_NAME,
content,
enabled: false,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Recheck the optimistic lock inside the updater

If another editor saves the configuration after the initial loadRawWorldbook conflict check but before this updateWorldbookWith callback runs, the callback receives that newer worldbook and unconditionally overwrites its target without prompting. The same race overwrites a newly created matching entry when entry_uid was null. Validate the target's expected state against the worldbook supplied to the updater before assigning content, so changes made during this gap are handled as conflicts.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant