Skip to content

fix(monitor): unify successor routing before writeback and receipt validation - #4145

Merged
huangruiteng merged 3 commits into
mainfrom
codex/monitor-successor-contract
Sep 10, 2026
Merged

fix(monitor): unify successor routing before writeback and receipt validation#4145
huangruiteng merged 3 commits into
mainfrom
codex/monitor-successor-contract

test(monitor): qualify routes against the production-scale fixture

935ebb8
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 9, 2026 in 2m 58s

Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
94.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 27 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use concise character class syntax '\d' instead of '[0-9]'. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DA&open=AaCGhS3nUKeUFGlo37DA&pullRequest=4145

Check warning on line 35 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Pass a message to the `Error` constructor. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DB&open=AaCGhS3nUKeUFGlo37DB&pullRequest=4145

Check failure on line 20 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37C-&open=AaCGhS3nUKeUFGlo37C-&pullRequest=4145

Check warning on line 47 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DD&open=AaCGhS3nUKeUFGlo37DD&pullRequest=4145

Check warning on line 48 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use concise character class syntax '\d' instead of '[0-9]'. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DE&open=AaCGhS3nUKeUFGlo37DE&pullRequest=4145

Check failure on line 79 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DF&open=AaCGhS3nUKeUFGlo37DF&pullRequest=4145

Check warning on line 23 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove duplicates in this character class. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37C_&open=AaCGhS3nUKeUFGlo37C_&pullRequest=4145

Check warning on line 41 in loopx/control_plane/scheduler/monitor_successor.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Pass a message to the `Error` constructor. 

See more on https://sonarcloud.io/project/issues?id=huangruiteng_loopx&issues=AaCGhS3nUKeUFGlo37DC&open=AaCGhS3nUKeUFGlo37DC&pullRequest=4145
View more details on SonarQubeCloud