NPE Risk: The non-existent ModelTypeEnum will cause the DevPilot settings page to fail to load properly.

openpilot-hub / devpilot-intellij

Your new coding buddy, designed exclusively for IntelliJ IDEA.

https://devpilot.zhongan.com

Apache License 2.0

593 stars 93 forks source link

NPE Risk: The non-existent ModelTypeEnum will cause the DevPilot settings page to fail to load properly. #8

Closed PerryZhao closed 7 months ago

PerryZhao commented 7 months ago

If a ModelType is removed in the future, users using that ModelType will not be able to load the configuration page normally. ModelServiceEnum and SessionTypeEnum also have the same problem

https://github.com/openpilot-hub/devpilot-intellij/blob/5fb2f5c19ec22622c816e69de1519149e285266d/src/main/java/com/zhongan/devpilot/enums/ModelServiceEnum.java#L23-L33

xiangtianyu commented 7 months ago

You are right, would you like to fix this problem?