Cannot seed tables without autoincrement ID field

[Swoorup]

Description

It appears that we cannot create a table without a Id column. Even if created the Id column has to be a autoincrement type.

To Reproduce

1. Generate any model like.

cargo loco generate model post name:string! url:string! icon:string!

2. Proceed to change the id type as either string or remove it.
3. Add seed data and run the seed task like cargo run task seed_data refresh:true
4. Observe the error.

Error: DB(Exec(SqlxError(Database(PgDatabaseError { severity: Error, code: "42703", message: "column \"id\" does not exist", detail: None, hint: None, position: Some(Original(70)), where: None, schema: None, table: None, column: None, data_type: None, constraint: None, file: Some("parse_relation.c"), line: Some(3656), routine: Some("errorMissingColumn") }))))

In the db logs, we see:

2024-11-28 19:38:29.131 UTC [375] STATEMENT:  SELECT setval(pg_get_serial_sequence('post', 'id'), COALESCE(MAX(id), 0) + 1, false) FROM post

Expected Behavior

Should run without any issues

Environment:

MacOSX

Additional Context

The issue is likely coming from this piece of logic here:

loco/src/db.rs

Lines 282 to 319 in e1b0392

	pub async fn reset_autoincrement(
	db_backend: DatabaseBackend,
	table_name: &str,
	db: &DatabaseConnection,
	) -> crate::Result<()> {
	match db_backend {
	DatabaseBackend::Postgres => {
	let query_str = format!(
	"SELECT setval(pg_get_serial_sequence('{table_name}', 'id'), COALESCE(MAX(id), 0) \
	+ 1, false) FROM {table_name}"
	);
	db.execute(Statement::from_sql_and_values(
	DatabaseBackend::Postgres,
	&query_str,
	vec![],
	))
	.await?;
	}
	DatabaseBackend::Sqlite => {
	let query_str = format!(
	"UPDATE sqlite_sequence SET seq = (SELECT MAX(id) FROM {table_name}) WHERE name = \
	'{table_name}'"
	);
	db.execute(Statement::from_sql_and_values(
	DatabaseBackend::Sqlite,
	&query_str,
	vec![],
	))
	.await?;
	}
	DatabaseBackend::MySql => {
	return Err(Error::Message(
	"Unsupported database backend: MySQL".to_string(),
	))
	}
	}
	Ok(())
	}

Should this logic change?