fix: address PR comments

Author: svetch

packages/cli/src/actions/db.ts

@@ -657,7 +657,7 @@ async function runPull(options: PullOptions) {
                 parseResult: { value: documentModel },
             } of docs) {
                 const zmodelSchema = await formatDocument(generator.generate(documentModel));
-                console.log(colors.blue(`Writing to ${uri.path}`));
+                console.log(colors.blue(`Writing to ${path.relative(process.cwd(), uri.fsPath).replace(/\\/g, '/')}`));
                 fs.writeFileSync(uri.fsPath, zmodelSchema);
             }
         }

packages/cli/src/actions/pull/provider/sqlite.ts

@@ -156,12 +156,12 @@ export const sqlite: IntrospectionProvider = {
                 const tableName = t.name;
                 const schema = '';
 
-                // Check if this table has autoincrement (via sqlite_sequence)
+                // Check if this table has autoincrement (detected by parsing the CREATE TABLE DDL)
                 const hasAutoIncrement = autoIncrementTables.has(tableName);
 
                 // PRAGMA table_xinfo: extended version of table_info that also includes hidden/generated columns.
                 // Returns one row per column with: cid (column index), name, type, notnull, dflt_value, pk.
-                // hidden: 0 = normal column, 1 = hidden/internal (e.g., rowid), 2 = generated/computed column.
+                // hidden: 0 = normal, 1 = hidden (virtual table), 2 = generated stored, 3 = generated virtual.
                 const columnsInfo = all<{
                     cid: number;
                     name: string;

packages/cli/src/actions/pull/utils.ts

@@ -72,11 +72,16 @@ export function getDatasource(model: Model) {
         ?.filter((s) => s !== undefined)) as string[] ||
         [];
 
+    const provider = getStringLiteral(
+        datasource.fields.find((f) => f.name === 'provider')?.value,
+    );
+    if (!provider) {
+        throw new CliError(`Datasource "${datasource.name}" is missing a "provider" field.`);
+    }
+
     return {
         name: datasource.name,
-        provider: getStringLiteral(
-            datasource.fields.find((f) => f.name === 'provider')?.value,
-        ) as DataSourceProviderType,
+        provider: provider as DataSourceProviderType,
         url,
         defaultSchema,
         schemas,