feat: Update major changes for defaults

[GeekMasher]

• i32 -> u64 for Primary and Foreign Keys
• Update Value defaults

[Copilot]

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• geekorm-core/src/builder/values/valsemver.rs: Evaluated as low risk

Comments suppressed due to low confidence (5)

geekorm-core/src/builder/values/mod.rs:77

• Changing the Integer variant from i32 to i64 may cause issues if other parts of the codebase expect i32 values. Ensure that all usages of Value::Integer are updated accordingly to handle i64 values.

Integer(i64),

geekorm-core/src/builder/values/mod.rs:80

• Changing the Boolean variant from i32 to u8 may cause issues if other parts of the codebase expect i32 values. Ensure that all usages of Value::Boolean are updated accordingly to handle u8 values.

Boolean(u8),

geekorm-core/src/builder/values/mod.rs:82

• Changing the Identifier variant from String to u64 may cause issues if other parts of the codebase expect String values. Ensure that all usages of Value::Identifier are updated accordingly to handle u64 values.

Identifier(u64),

geekorm-core/src/backends/libsql.rs:239

• The error message 'NoRowsFound' should include the query string to provide more context.

return Err(crate::Error::NoRowsFound(query.to_string()));

geekorm-core/src/backends/libsql.rs:398

• This change assumes that libsql::Value::Integer can hold a u64 value, which may not be true. Ensure the type compatibility or handle potential overflows.

libsql::Value::Integer(value) => Value::Integer(value),

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more