Alanxtl opened a new issue, #3204:
URL: https://github.com/apache/dubbo-go/issues/3204

   > ### 第一阶段:单测先行 (Fix Unit Tests) 🧪
   > 
   > **目标**:优先重构单测代码中的配置初始化逻辑。解决测试环境下的配置注入问题,为后续改动提供验证基准。
   > 
   > - [ ] **路由与集群单测**
   >   - [ ] `cluster/router/condition/router_test.go`
   > - [ ] **配置中心单测**
   >   - [ ] `config_center/apollo/impl_test.go`
   > - [ ] **协议与过滤器单测**
   >   - [ ] `filter/graceful_shutdown/consumer_filter_test.go`
   >   - [ ] `filter/graceful_shutdown/provider_filter_test.go`
   >   - [x] `protocol/dubbo3/common_test.go`
   >   - [x] `protocol/grpc/grpc_protocol_test.go`
   >   - [x] `protocol/jsonrpc/jsonrpc_protocol_test.go`
   >   - [x] `protocol/rest/rest_invoker_test.go`
   >   - [x] `protocol/rest/rest_protocol_test.go`
   > - [ ] **核心组件单测**
   >   - [ ] `remoting/nacos/builder_test.go`
   >   - [ ] `server/action_test.go`
   >   - [ ] `server/compat_test.go`
   >   - [ ] `remoting/getty/getty_client_test.go`
   >   - [ ] `remoting/getty/getty_server_test.go`
   > 
   > ------
   > 
   > ### 第二阶段:核心底层逻辑迁移 (Core Logic - Config Only Files) 🏗️
   > 
   > **目标**:在修改 CLI 工具前,必须先将标记为 `config only` 的非单测核心逻辑迁移至 
`global`。这是确保生成代码能够正常运行的前提。
   > 
   > - [ ] **路由配置逻辑重构**
   >   - [ ] `cluster/router/condition/dynamic_router.go`
   >   - [ ] `cluster/router/condition/route.go`
   >   - [ ] `cluster/router/script/router.go`
   > - [ ] **配置中心实现重构**
   >   - [ ] `config_center/zookeeper/impl.go`
   > - [ ] **流量治理插件重构**
   >   - [ ] `filter/hystrix/filter.go`
   >   - [ ] `filter/polaris/limit/limiter.go`
   > - [ ] **协议内部实现迁移**
   >   - [ ] `protocol/dubbo3/internal/client.go`
   >   - [ ] `protocol/dubbo3/internal/server.go`
   >   - [ ] `protocol/grpc/internal/helloworld/client.go`
   >   - [ ] `protocol/grpc/internal/routeguide/client.go`
   >   - [ ] `protocol/triple/health/healthServer.go`
   >   - [ ] `protocol/triple/reflection/serverreflection.go`
   > 
   > ------
   > 
   > ### 第三阶段:CLI 工具与项目模板迁移 (CLI & Templates) 🛠️
   > 
   > **目标**:当底层逻辑就绪后,同步更新 `dubbogo-cli` 及其内置模板,确保新生成的项目天然符合 `global` 规范。
   > 
   > - [ ] **生成器逻辑更新**
   >   - [ ] `tools/dubbogo-cli/generator/application/cmd.go`
   >   - [ ] `tools/dubbogo-cli/generator/application/pkg.go`
   >   - [ ] `tools/dubbogo-cli/generator/sample/gen_client.go`
   >   - [ ] `tools/dubbogo-cli/generator/sample/gen_server.go`
   > - [ ] **项目骨架模板更新**
   >   - [ ] `tools/dubbogo-cli/cmd/testGenCode/template/newApp/cmd/app.go`
   >   - [ ] 
`tools/dubbogo-cli/cmd/testGenCode/template/newApp/pkg/service/service.go`
   >   - [ ] 
`tools/dubbogo-cli/cmd/testGenCode/template/newDemo/go-client/cmd/client.go`
   >   - [ ] 
`tools/dubbogo-cli/cmd/testGenCode/template/newDemo/go-server/cmd/server.go`
   > 
   > ------
   > 
   > ### 第四阶段:全面切流与清理 (Full Migration & Cleanup) 🚀
   > 
   > **目标**:修改剩余的 `config+global` 混合引用文件,彻底移除对旧 `config` 包的依赖,最后清理 `compat.go`。
   > 
   > - [ ] **入口与选项文件迁移**
   >   - [ ] `client/action.go` / `server/action.go`
   >   - [ ] `options.go` / `client/options.go` / `server/options.go`
   > - [ ] **核心协议层最终迁移**
   >   - [ ] `protocol/dubbo3/dubbo3_invoker.go` / `dubbo3_protocol.go`
   >   - [ ] `protocol/triple/server.go` / `dubbo3_invoker.go`
   >   - [ ] `protocol/grpc/client.go` / `server.go`
   > - [ ] **注册中心与通信层迁移**
   >   - [ ] `registry/directory/directory.go` / `protocol.go`
   >   - [ ] `remoting/getty/getty_client.go` / `getty_server.go`
   > - [ ] **清理兼容层**
   >   - [ ] `compat.go` / `client/compat.go` / `server/compat.go`
   > 
   > 
   > 
   >  
   
    _Originally posted by @nanjiek in 
[#2741](https://github.com/apache/dubbo-go/issues/2741#issuecomment-3834875233)_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to