Files
upstream-twenty/packages/twenty-oxlint-rules/rules/no-data-mutation-in-fast-instance-command.ts
upstream-mirror e7da97194a
Some checks failed
App Prod-Parity E2E Dispatch / dispatch (push) Failing after 10s
CD Storybook UI / build (push) Failing after 3m25s
CD Storybook UI / dispatch (push) Has been skipped
CD deploy main / website-changed-files (push) Successful in 18m8s
CI Agent Skills / changed-files-check (push) Successful in 18m9s
CI Create App E2E minimal / changed-files-check (push) Successful in 24s
CI Create App / changed-files-check (push) Successful in 26s
CI Docs / changed-files-check (push) Successful in 26s
CI Emails / changed-files-check (push) Successful in 28s
CI Example App Hello World / changed-files-check (push) Successful in 23s
CI Front / changed-files-check (push) Has been skipped
CI Example App Postcard / changed-files-check (push) Successful in 27s
CI UI / changed-files-check (push) Has been skipped
CI Utils Test / changed-files-check (push) Successful in 45s
CI Twenty Apps / discover (push) Successful in 1m3s
CD deploy main / deploy-main (push) Failing after 3s
CI Agent Skills / agent-skills-validate (push) Has been skipped
CI Create App / create-app-test (lint) (push) Has been skipped
CI Emails / emails-test (push) Has been skipped
CI Front / front-task (lingui:compile) (push) Has been skipped
CI Front / front-task (lint) (push) Has been skipped
CI Example App Postcard / example-app-postcard (push) Has been skipped
CI UI / ui-task (lint) (push) Has been skipped
CI UI / ui-task (pack-size) (push) Has been skipped
CI UI / ui-task (size) (push) Has been skipped
CI UI / ui-task (test) (push) Has been skipped
CI UI / ui-task (typecheck) (push) Has been skipped
CI Utils Test / utils-test (push) Has been skipped
CI Twenty Apps / ${{ matrix.app.name }} (push) Has been skipped
CI Emails / ci-emails-status-check (push) Successful in 2s
CI Example App Hello World / ci-example-app-hello-world-status-check (push) Successful in 2s
CI Example App Postcard / ci-example-app-postcard-status-check (push) Successful in 3s
CI UI / ui-sb-test (push) Has been skipped
CI Utils Test / ci-utils-test-status-check (push) Successful in 2s
CI Twenty Apps / ci-twenty-apps-status-check (push) Successful in 2s
CI UI / ci-ui-status-check (push) Successful in 2s
CI Desktop / desktop (push) Failing after 1m8s
Push docs to Crowdin / Push documentation to Crowdin (push) Failing after 55s
Push translations to Crowdin / Extract and upload translations (push) Failing after 5s
Push website translations to Crowdin / Extract and upload website translations (push) Failing after 5s
CI Create App / create-app-test (test) (push) Has been skipped
CI Create App / create-app-test (typecheck) (push) Has been skipped
CI Docs / docs-lint (push) Has been skipped
CI Example App Hello World / example-app-hello-world (push) Has been skipped
CI Front / front-task (lingui:extract) (push) Has been skipped
CI Front / front-task (test) (push) Has been skipped
CI Front / front-task (typecheck) (push) Has been skipped
CI UI / ui-sb-build (push) Failing after 1m32s
CI Twenty Apps / ${{ matrix.app.name }} (${{ matrix.server-source }}) (push) Has been skipped
CI Agent Skills / agent-skills-publish (push) Has been skipped
CI Create App / ci-create-app-status-check (push) Successful in 2s
CI Agent Skills / agent-skills-remote-source (push) Has been skipped
CI Create App E2E minimal / create-app-e2e-minimal (push) Has been cancelled
CI E2E Main / e2e-test (push) Has been cancelled
CI Front / front-sb-build (push) Has been cancelled
CI Front / front-build (push) Has been cancelled
CI Create App E2E minimal / ci-create-app-e2e-minimal-status-check (push) Has been cancelled
CI E2E Main / ci-e2e-main-status-check (push) Has been cancelled
CI E2E Main / notify-main-ci-failure (push) Has been cancelled
CI E2E Main / qa-scout-comment (push) Has been cancelled
CI Front / front-sb-test (push) Has been cancelled
CI Front / front-sb-screenshots (push) Has been cancelled
CI Front / ci-front-status-check (push) Has been cancelled
DPA Sub-Processor Sync / sync-subprocessors (push) Failing after 3s
AI Catalog Sync / sync-catalog (push) Failing after 2s
snapshot of https://github.com/twentyhq/twenty.git (main, 2026-09-17; blobs > 131072B omitted)
2026-09-17 23:28:13 +05:00

144 lines
3.6 KiB
TypeScript

import { defineRule } from '@oxlint/plugins';
export const RULE_NAME = 'no-data-mutation-in-fast-instance-command';
const UPGRADE_COMMAND_MARKER = 'upgrade-version-command/';
const FAST_INSTANCE_COMMAND_SEGMENT = 'instance-command-fast-';
const SKIPPED_FILE_REGEX = /\.(spec|test)\.ts$/;
const DATA_MUTATION_STATEMENT_REGEX = /^(UPDATE|INSERT|DELETE|MERGE)\b/i;
const CTE_DATA_MUTATION_REGEX =
/^WITH\b[\s\S]*[()]\s*(UPDATE|INSERT|DELETE|MERGE)\b/i;
const isFastInstanceCommandFile = (filename: string): boolean => {
const markerIndex = filename.indexOf(UPGRADE_COMMAND_MARKER);
if (markerIndex === -1) {
return false;
}
const basename =
filename
.slice(markerIndex + UPGRADE_COMMAND_MARKER.length)
.split('/')
.pop() ?? '';
if (SKIPPED_FILE_REGEX.test(basename)) {
return false;
}
return basename.includes(FAST_INSTANCE_COMMAND_SEGMENT);
};
const isInsideDownMethod = (node: any): boolean => {
let current = node.parent;
while (current) {
if (
(current.type === 'MethodDefinition' ||
current.type === 'PropertyDefinition') &&
current.key?.type === 'Identifier' &&
current.key.name === 'down'
) {
return true;
}
current = current.parent;
}
return false;
};
const getSqlText = (argument: any): string | null => {
if (argument.type === 'Literal' && typeof argument.value === 'string') {
return argument.value;
}
if (argument.type === 'TemplateLiteral') {
return argument.quasis.map((quasi: any) => quasi.value.raw).join(' ');
}
return null;
};
const findDataMutationKeyword = (sql: string): string | null => {
const normalized = sql
.replace(/--[^\n]*/g, '')
.replace(/\/\*[\s\S]*?\*\//g, '')
.replace(/'(?:[^']|'')*'/g, "''")
.replace(/"(?:[^"]|"")*"/g, '""');
for (const statement of normalized.split(';')) {
const trimmed = statement.trim();
const match =
trimmed.match(DATA_MUTATION_STATEMENT_REGEX) ??
trimmed.match(CTE_DATA_MUTATION_REGEX);
if (match) {
return match[1].toUpperCase();
}
}
return null;
};
export const rule = defineRule({
meta: {
type: 'problem',
docs: {
description:
'Disallow data mutations (UPDATE/INSERT/DELETE) in a fast instance command outside down(); backfills belong in a slow instance command',
},
schema: [],
messages: {
dataMutationInFastInstanceCommand:
"Fast instance commands must not run data mutations outside down() (found '{{ keyword }}'). A bulk write held in the same transaction as ADD/ALTER COLUMN keeps an ACCESS EXCLUSIVE lock and can stall reads during the deploy. Put backfills in a slow instance command's runDataMigration(); rollback writes belong in down().",
},
},
create: (context) => {
if (!isFastInstanceCommandFile(context.filename)) {
return {};
}
return {
CallExpression: (node: any) => {
const callee = node.callee;
if (
callee?.type !== 'MemberExpression' ||
callee.property?.type !== 'Identifier' ||
callee.property.name !== 'query'
) {
return;
}
if (isInsideDownMethod(node)) {
return;
}
const sqlArgument = node.arguments?.[0];
if (!sqlArgument) {
return;
}
const sql = getSqlText(sqlArgument);
if (sql === null) {
return;
}
const keyword = findDataMutationKeyword(sql);
if (keyword) {
context.report({
node: sqlArgument,
messageId: 'dataMutationInFastInstanceCommand',
data: { keyword },
});
}
},
};
},
});