diff --git a/.eslintrc.js b/.eslintrc.js index 6d8e085182..e85e0d768c 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -19,6 +19,7 @@ module.exports = { 'e2e/playwright-report/**/*', 'packages/data-provider/types/**/*', 'packages/data-provider/dist/**/*', + 'packages/data-provider/test_bundle/**/*', 'data-node/**/*', 'meili_data/**/*', 'node_modules/**/*', diff --git a/.github/workflows/backend-review.yml b/.github/workflows/backend-review.yml index 2d5cf387be..9f8bd7b530 100644 --- a/.github/workflows/backend-review.yml +++ b/.github/workflows/backend-review.yml @@ -35,6 +35,21 @@ jobs: - name: Install Data Provider run: npm run build:data-provider + + - name: Create empty auth.json file + run: | + mkdir -p api/data + echo '{}' > api/data/auth.json + + - name: Check for Circular dependency in rollup + working-directory: ./packages/data-provider + run: | + output=$(npm run rollup:api) + echo "$output" + if echo "$output" | grep -q "Circular dependency"; then + echo "Error: Circular dependency detected!" + exit 1 + fi - name: Run unit tests run: cd api && npm run test:ci diff --git a/api/utils/azureUtils.js b/api/utils/azureUtils.js index fe8a34d6d6..27396a8fc5 100644 --- a/api/utils/azureUtils.js +++ b/api/utils/azureUtils.js @@ -6,7 +6,7 @@ const { isEnabled } = require('~/server/utils'); * @returns {string} The sanitized model name. */ const sanitizeModelName = (modelName) => { - // Replace periods with empty strings and other disallowed characters as needed + // Replace periods with empty strings and other disallowed characters as needed. return modelName.replace(/\./g, ''); };