bmcgo gen工具优化建议

交流互助区
1

需求开发中，遇到使用gen工具自动生成代码后还需要手动调整gen的代码才能保证组件功能正常或通过社区CI的普通检查的问题。

  1. pcie_device组件的代码进行gen之后，发现被删去了CpuBusInfo相关的部分gen代码

model.json中的配置（未修改）

"CpuBusInfo": {
        "tableName": "t_cpu_bus_info",
        "privilege": ["ConfigureSelf"],
        "properties": {
            "SocketID": {
                "baseType": "U8",
                "default": 0,
                "primaryKey": true
            },
            "SysId": {
                "baseType": "U8",
                "default": 1
            },
            "BusBaseAddr": {
                "baseType": "U8",
                "default": 0
            },
            "BusSize": {
                "baseType": "U8",
                "default": 0
            }
        }
    }

gen后缺少的代码

image
image1752×150 17.3 KB

image
image1318×247 16.3 KB

对相关功能可能产生影响

  1. 建议gen工具自动检查生成代码行宽是否超过120，减少为通过ci检查而进行的手动修改动作
Lua和C混合组件的新增代码覆盖率检查怎么处理
2

+1

3
  1. 自动生成代码策略，没有配置tabletype不会自动生成相关代码
  2. 行宽未限制120目前认为是vscode-lua-format的bug，暂时只能通过手动修改
4

1.这个策略是否相较于旧的策略有变动，因为从原先gen下的代码看，旧的策略是可以生成这相关的代码的（在gen下的代码没有被手动修改的前提下）；如策略有更新，是否有评估过其兼容性，目前从结果上来看可能是存在一定问题的
2.目前有修复计划吗，或者尝试使用其他工具，手动修改不是长久之计呀

5
  1. 当前自动生成代码已通过版本进行区分，自动生成代码策略发生变更时会通过版本区分，具体在service.json下 codegen_policy的version关键字，能够约束自动生成代码不随版本出现预期外的变更
  2. 行宽超过120这个是来自配置文件vscode-lua-format的bug，目前只能等待官方社区更新