ADORSYS-GIS / ADORSYS-GIS/lightbridge-governance

[Ticket]: Split app/governance-ctl/src/sync.rs (1178 LoC)

未关闭
#178 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
ticket
主要语言
Rust
星标
1
派生
2
平均合并
13 小时 13 分钟
30 天内合并 PR
110

描述

### Type

Refactor

### Summary

`app/governance-ctl/src/sync.rs` is 1178 lines. Split it along its real seams to bring every resulting file under 200.

### Intent

It is one of the largest files in the repository and, at this size, is skimmed rather than reviewed.

### Source of truth (links)

- #162 (parent epic), #173 (baseline)

### Acceptance Criteria

1. No resulting file exceeds 200 LoC.
2. **Behaviour is unchanged** — the existing tests pass before and after, unmodified.
3. No test is weakened, deleted or made less specific to accommodate the split.
4. Module boundaries follow a real seam, not an arbitrary line count.
5. Public API unchanged, or the change is called out explicitly.

### Out of Scope

Behaviour changes of any kind. This is a pure refactor.

### Technical Context

The copilot-sync command. Seams: fetch, parse, upsert, manifest bookkeeping.

### Risks

⚠️ A shrinking file that also shrinks its assertions is a regression wearing a refactor's clothes. Reviewers should diff the test count and the assertion count, not just the line count.

### Test Plan

The existing suite, unmodified, before and after. If a test needed changing, say why in the PR — that is the signal worth reviewing.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。