Skip to content

fix(scripts): 拆分 compose 生成器 main 消除 Sonar S3776 - #506

Merged
AlianBlank merged 1 commit into
mainfrom
feature/auto-server-source-worker-run-7811-20260922T1240
Sep 22, 2026
Merged

AlianBlank merged 1 commit into
mainfrom
feature/auto-server-source-worker-run-7811-20260922T1240

Conversation

@AlianBlank

Copy link
Copy Markdown
Collaborator

Linear: GFX-864

把 generate-docker-compose-multi.py 的 main() 六段职责抽取为
expand_role_instances / render_mongo_service / render_instance_service_body /
render_launcher_services / verify_outputs / write_outputs 六个模块级辅助
函数,认知复杂度 31 降至 1(辅助函数最高 11)。字符串字面量与拼接顺序
原样搬移,生成产物字节级不变,--check 与 CLI 契约保持不变。

Linear: GFX-864
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 21 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: aa3ba811-29f0-466b-9945-d24036d2c7b6

📥 Commits

Reviewing files that changed from the base of the PR and between bb218a5 and 37dd5d3.

📒 Files selected for processing (1)
  • scripts/multi/generate-docker-compose-multi.py

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@AlianBlank
AlianBlank merged commit 6ee1e60 into main Sep 22, 2026
2 of 3 checks passed
@AlianBlank
AlianBlank deleted the feature/auto-server-source-worker-run-7811-20260922T1240 branch September 22, 2026 12:57
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