diff --git a/.changeset/sixty-eels-camp.md b/.changeset/sixty-eels-camp.md new file mode 100644 index 0000000000..7fbd0d9614 --- /dev/null +++ b/.changeset/sixty-eels-camp.md @@ -0,0 +1,5 @@ +--- +"@astrojs/db": patch +--- + +Prevent runtime from importing core code diff --git a/.eslintrc.cjs b/.eslintrc.cjs index 552033711b..f3401b2d26 100644 --- a/.eslintrc.cjs +++ b/.eslintrc.cjs @@ -136,5 +136,23 @@ module.exports = { 'arrow-body-style': ['error', 'never'], }, }, + + { + files: ['packages/db/src/runtime/**/*.ts'], + rules: { + 'no-restricted-imports': 'off', + '@typescript-eslint/no-restricted-imports': [ + 'error', + { + patterns: [ + { + group: ['../core/*'], + allowTypeImports: true, + } + ] + } + ] + } + } ], }; diff --git a/packages/db/src/core/errors.ts b/packages/db/src/core/errors.ts index 198c86b787..64ff271d80 100644 --- a/packages/db/src/core/errors.ts +++ b/packages/db/src/core/errors.ts @@ -37,46 +37,14 @@ export const SHELL_QUERY_MISSING_ERROR = `${red( '▶ Please provide a query to execute using the --query flag.' )}\n`; -export const SEED_ERROR = (error: string) => { - return `${red(`Error while seeding database:`)}\n\n${error}`; -}; - export const EXEC_ERROR = (error: string) => { return `${red(`Error while executing file:`)}\n\n${error}`; }; -export const SEED_DEFAULT_EXPORT_ERROR = (fileName: string) => { - return SEED_ERROR(`Missing default function export in ${bold(fileName)}`); -}; - export const EXEC_DEFAULT_EXPORT_ERROR = (fileName: string) => { return EXEC_ERROR(`Missing default function export in ${bold(fileName)}`); }; -export const REFERENCE_DNE_ERROR = (columnName: string) => { - return `Column ${bold( - columnName - )} references a table that does not exist. Did you apply the referenced table to the \`tables\` object in your db config?`; -}; - -export const FOREIGN_KEY_DNE_ERROR = (tableName: string) => { - return `Table ${bold( - tableName - )} references a table that does not exist. Did you apply the referenced table to the \`tables\` object in your db config?`; -}; - -export const FOREIGN_KEY_REFERENCES_LENGTH_ERROR = (tableName: string) => { - return `Foreign key on ${bold( - tableName - )} is misconfigured. \`columns\` and \`references\` must be the same length.`; -}; - -export const FOREIGN_KEY_REFERENCES_EMPTY_ERROR = (tableName: string) => { - return `Foreign key on ${bold( - tableName - )} is misconfigured. \`references\` array cannot be empty.`; -}; - export const INTEGRATION_TABLE_CONFLICT_ERROR = ( integrationName: string, tableName: string, diff --git a/packages/db/src/runtime/errors.ts b/packages/db/src/runtime/errors.ts new file mode 100644 index 0000000000..2026e57e97 --- /dev/null +++ b/packages/db/src/runtime/errors.ts @@ -0,0 +1,33 @@ +import { bold, red } from 'kleur/colors'; + +export const FOREIGN_KEY_DNE_ERROR = (tableName: string) => { + return `Table ${bold( + tableName + )} references a table that does not exist. Did you apply the referenced table to the \`tables\` object in your db config?`; +}; + +export const FOREIGN_KEY_REFERENCES_LENGTH_ERROR = (tableName: string) => { + return `Foreign key on ${bold( + tableName + )} is misconfigured. \`columns\` and \`references\` must be the same length.`; +}; + +export const FOREIGN_KEY_REFERENCES_EMPTY_ERROR = (tableName: string) => { + return `Foreign key on ${bold( + tableName + )} is misconfigured. \`references\` array cannot be empty.`; +}; + +export const REFERENCE_DNE_ERROR = (columnName: string) => { + return `Column ${bold( + columnName + )} references a table that does not exist. Did you apply the referenced table to the \`tables\` object in your db config?`; +}; + +export const SEED_ERROR = (error: string) => { + return `${red(`Error while seeding database:`)}\n\n${error}`; +}; + +export const SEED_DEFAULT_EXPORT_ERROR = (fileName: string) => { + return SEED_ERROR(`Missing default function export in ${bold(fileName)}`); +}; diff --git a/packages/db/src/runtime/queries.ts b/packages/db/src/runtime/queries.ts index 08e2f5e29d..01fefb7457 100644 --- a/packages/db/src/runtime/queries.ts +++ b/packages/db/src/runtime/queries.ts @@ -6,7 +6,7 @@ import { FOREIGN_KEY_REFERENCES_EMPTY_ERROR, FOREIGN_KEY_REFERENCES_LENGTH_ERROR, REFERENCE_DNE_ERROR, -} from '../core/errors.js'; +} from './errors.js'; import type { BooleanColumn, ColumnType, diff --git a/packages/db/src/runtime/seed-local.ts b/packages/db/src/runtime/seed-local.ts index f00e44ea0b..3e3a4eacea 100644 --- a/packages/db/src/runtime/seed-local.ts +++ b/packages/db/src/runtime/seed-local.ts @@ -2,7 +2,7 @@ import { LibsqlError } from '@libsql/client'; import { type SQL, sql } from 'drizzle-orm'; import type { LibSQLDatabase } from 'drizzle-orm/libsql'; import { SQLiteAsyncDialect } from 'drizzle-orm/sqlite-core'; -import { SEED_DEFAULT_EXPORT_ERROR, SEED_ERROR } from '../core/errors.js'; +import { SEED_DEFAULT_EXPORT_ERROR, SEED_ERROR } from './errors.js'; import { type DBTables } from '../core/types.js'; import { getCreateIndexQueries, getCreateTableQuery } from './queries.js';