Sqlite buildDataType drops the column size when decimalDigits is null (unreachable dead branch)

[Aias00]

Chat2DB Edition

Chat2DB Community

Chat2DB Version

5.3.0

Deployment

Web

Operating System

macOS

Operating System Version

12

Database and Version

SQLite

Problem Summary

SqliteColumnTypeEnum.buildDataType guards with if (columnSize == null || decimalDigits == null) return columnType. The || decimalDigits == null causes an early return for sized columns without scale (VARCHAR(255), INTEGER(10)) — the size is lost. The size-only branch at line 152 is unreachable dead code because line 149 already returned. Every sibling guards on columnSize == null only.

Expected Behavior

Guard on columnSize == null only, making the size-only branch reachable.

Actual Behavior

SqliteColumnTypeEnum.java:149 — if (columnSize == null || decimalDigits == null) return columnType.

Impact

Limited or cosmetic impact

Workaround

N/A

Submission Checklist

• I reproduced this on the selected Chat2DB edition.
• I searched existing issues for duplicates.
• I removed credentials and private data.

[github-actions]

Thanks for the precise analysis of SqliteColumnTypeEnum.buildDataType; I understand that the decimalDigits == null guard drops sizes such as VARCHAR(255) and leaves the size-only branch unreachable.

The required Steps to Reproduce field is missing. Please add the smallest concrete invocation or regression test, including the columnType, columnSize, and decimalDigits inputs and the actual versus expected returned string. That will give maintainers a direct way to validate the behavior while reviewing the linked fix in #2170.

Generated by AI Issue Assistant for #2167 · 4.42 AIC · ⌖ 2.69 AIC · ⊞ 32.8K · ◷