Skip to content

refactor(coordination): make Goal Channel ownership provider-aware - #4316

Merged
huangruiteng merged 3 commits into
mainfrom
codex/canonical-coordination-observation
Sep 14, 2026
Merged

huangruiteng merged 3 commits into
mainfrom
codex/canonical-coordination-observation

fix(coordination): harden ownership observation redaction

412126b
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 14, 2026 in 2m 29s

Annotations

Check warning on line 146 in loopx/control_plane/effect_runtime_handlers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'/home/runner/work/loopx/loopx/loopx/control_plane/coordination/local_authority_runtime.ts' imported multiple times.

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

Check warning on line 85 in loopx/control_plane/coordination/ownership_observation.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

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

Check warning on line 54 in tests/control_plane/test_canonical_goal_channel_coordination.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

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

Check failure on line 10 in loopx/control_plane/goals/coordination_observation.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 4 in loopx/control_plane/effect_runtime_handlers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'/home/runner/work/loopx/loopx/loopx/control_plane/coordination/local_authority_runtime.ts' imported multiple times.

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

Check failure on line 40 in loopx/control_plane/coordination/ownership_observation.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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