Skip to content

Staging #1317

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Staging #1317

wants to merge 7 commits into from

Conversation

MH4GF
Copy link
Member

@MH4GF MH4GF commented Apr 11, 2025

Issue

  • resolve:

Why is this change needed?

What would you like reviewers to focus on?

Testing Verification

What was done

pr_agent:summary

Detailed Changes

pr_agent:walkthrough

Additional Notes

Copy link

changeset-bot bot commented Apr 11, 2025

⚠️ No Changeset found

Latest commit: 8e7e114

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Apr 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
liam-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2025 5:45am
liam-erd-sample ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2025 5:45am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
liam-docs ⬜️ Ignored (Inspect) Visit Preview Apr 25, 2025 5:45am

Copy link

supabase bot commented Apr 11, 2025

Updates to Preview Branch (staging) ↗︎

Deployments Status Updated
Database Fri, 25 Apr 2025 05:41:45 UTC
Services Fri, 25 Apr 2025 05:41:45 UTC
APIs Fri, 25 Apr 2025 05:41:45 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations ⚠️ Fri, 25 Apr 2025 05:41:45 UTC
Migrations Fri, 25 Apr 2025 05:41:45 UTC
Seeding Fri, 25 Apr 2025 05:41:45 UTC
Edge Functions Fri, 25 Apr 2025 05:41:45 UTC

⚠️ Warning — You can configure a persistent branch with config.toml:
[remotes.staging]
project_id = "zftuappjeluwcuaerlfe"


View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

Copy link
Contributor

github-actions bot commented Apr 11, 2025

Copy link
Contributor

qodo-merge-pro-for-open-source bot commented Apr 11, 2025

CI Feedback 🧐

(Feedback updated until commit 0b3d4f7)

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: frontend-ci

Failed stage: Check for diff in generated types and schema.sql [❌]

Failure summary:

The action failed because the generated types and schema.sql files differ from the committed files,
as indicated in line 1763: "Generated types and schema.sql differ from committed files."

The log shows differences in foreign key names that were truncated:

  • Line 1750-1751: 'overall_review_knowledge_suggestion_ma_knowledge_suggestion_id_fkey' changed to
    'overall_review_knowledge_suggestion_mapping_knowledge_suggestio'
  • Line 1758-1759: 'overall_review_knowledge_suggestion_ma_overall_review_id_fkey' changed to
    'overall_review_knowledge_suggestion_mapping_overall_review_id_f'

    These changes need to be committed to the repository for the check to pass.

  • Relevant error logs:
    1:  ##[group]Operating System
    2:  Ubuntu
    ...
    
    158:  �[36;1mpnpm install --frozen-lockfile --prefer-offline�[0m
    159:  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
    160:  env:
    161:  PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin
    162:  ##[endgroup]
    163:  Scope: all 17 workspace projects
    164:  Lockfile is up to date, resolution step is skipped
    165:  Progress: resolved 1, reused 0, downloaded 0, added 0
    166:  Packages: +1567
    167:  ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
    168:  Progress: resolved 1567, reused 826, downloaded 0, added 0
    169:  Progress: resolved 1567, reused 1554, downloaded 0, added 330
    170:  Progress: resolved 1567, reused 1554, downloaded 0, added 796
    171:  Progress: resolved 1567, reused 1554, downloaded 0, added 1357
    172:  Progress: resolved 1567, reused 1554, downloaded 0, added 1567, done
    173:  WARN  Failed to create bin at /home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/.bin/liam. ENOENT: no such file or directory, open '/home/runner/work/liam/liam/frontend/packages/cli/dist-cli/bin/cli.js'
    174:  devDependencies:
    ...
    
    187:  │                                                                              │
    188:  │   Ignored build scripts: @biomejs/biome, @bundled-es-modules/glob,           │
    189:  │   @depot/cli, @prisma/client, @prisma/engines, @sentry/cli, core-js-pure,    │
    190:  │   esbuild, protobufjs, sharp, style-dictionary.                              │
    191:  │   Run "pnpm approve-builds" to pick which dependencies should be allowed     │
    192:  │   to run scripts.                                                            │
    193:  │                                                                              │
    194:  ╰──────────────────────────────────────────────────────────────────────────────╯
    195:  frontend/apps/docs postinstall$ fumadocs-mdx
    196:  frontend/packages/jobs postinstall$ cp ../db-structure/node_modules/@ruby/prism/src/prism.wasm prism.wasm
    197:  frontend/packages/jobs postinstall: Done
    198:  frontend/apps/docs postinstall: [MDX] types generated
    199:  frontend/apps/docs postinstall: Done
    200:  frontend/apps/app postinstall$ cp ../../packages/db-structure/node_modules/@ruby/prism/src/prism.wasm prism.wasm
    201:  frontend/apps/app postinstall: Done
    202:  WARN  Failed to create bin at /home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/.bin/liam. ENOENT: no such file or directory, open '/home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/@liam-hq/cli/dist-cli/bin/cli.js'
    203:  Done in 6.9s using pnpm v10.8.1
    ...
    
    940:  ##[group]@liam-hq/erd-core:gen
    941:  cache miss, executing 13b77f17012f2d10
    942:  > @liam-hq/erd-core@0.1.7 gen /home/runner/work/liam/liam/frontend/packages/erd-core
    943:  > concurrently "pnpm:gen:*"
    944:  [css] 
    945:  [css] > @liam-hq/erd-core@0.1.7 gen:css /home/runner/work/liam/liam/frontend/packages/erd-core
    946:  [css] > tcm src
    947:  [css] 
    948:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/styles/variables.css.d.ts
    949:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ERDRenderer.module.css.d.ts
    950:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDContent/ERDContent.module.css.d.ts
    951:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/RelationshipEdgeParticleMarker/RelationshipEdgeParticleMarker.module.css.d.ts
    952:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/LeftPane/LeftPane.module.css.d.ts
    953:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/TableDetailDrawer/TableDetailDrawer.module.css.d.ts
    954:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/Toolbar/DesktopToolbar.module.css.d.ts
    955:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/ParseErrorDisplay.module.css.d.ts
    956:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/ErrorDisplay.module.css.d.ts
    957:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/NetworkErrorDisplay.module.css.d.ts
    958:  [css] Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/CardinalityMarkers/CardinalityMarkers.module.css.d.ts
    ...
    
    1125:  �[32m✓�[39m app/(app)/app/projects/[projectId]/ref/[branchOrCommit]/docs/getGitHubDocFilePaths.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m | �[22m�[33m1 skipped�[39m�[2m)�[22m�[32m 112�[2mms�[22m�[39m
    1126:  �[90mstderr�[2m | app/api/webhook/github/utils/__tests__/checkSchemaChanges.test.ts�[2m > �[22m�[2mcheckSchemaChanges�[2m > �[22m�[2mshould return false if no files match the schema paths
    1127:  �[22m�[39mNo schema path found for project 100: {"code":"22P02","details":null,"hint":null,"message":"invalid input syntax for type uuid: \"100\""}
    1128:  �[32m✓�[39m app/api/webhook/github/utils/__tests__/checkSchemaChanges.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m | �[22m�[33m1 skipped�[39m�[2m)�[22m�[32m 184�[2mms�[22m�[39m
    1129:  �[2m�[90m↓�[39m�[22m features/projects/pages/ProjectsPage/getProjects.test.ts �[2m(�[22m�[2m1 test�[22m�[2m | �[22m�[33m1 skipped�[39m�[2m)�[22m
    1130:  �[2m Test Files �[22m �[1m�[32m3 passed�[39m�[22m�[2m | �[22m�[33m1 skipped�[39m�[90m (4)�[39m
    1131:  �[2m      Tests �[22m �[1m�[32m4 passed�[39m�[22m�[2m | �[22m�[33m4 skipped�[39m�[90m (8)�[39m
    1132:  �[2m   Start at �[22m 05:25:34
    1133:  �[2m   Duration �[22m 2.55s�[2m (transform 527ms, setup 421ms, collect 1.86s, tests 518ms, environment 6ms, prepare 1.25s)�[22m
    1134:  ##[endgroup]
    1135:  ##[group]@liam-hq/prompt-test:test
    1136:  cache miss, executing d74c556319c97fe4
    1137:  > @liam-hq/prompt-test@0.1.0 test /home/runner/work/liam/liam/frontend/packages/prompt-test
    1138:  > tsx src/index.ts
    1139:  Langfuse secret key was not passed to constructor or not set as 'LANGFUSE_SECRET_KEY' environment variable. No observability data will be sent to Langfuse.
    1140:  [Langfuse SDK] 401: Unauthorized. Please check your public/private host settings. Refer to our installation and setup guide: https://langfuse.com/docs/sdk/typescript/guide for details on SDK configuration. Make sure to keep your SDK updated, refer to https://www.npmjs.com/package/langfuse for details. Error details: LangfuseFetchHttpError: HTTP error while fetching Langfuse: 401 and body: {"message":"Invalid authorization header"}
    1141:  [Langfuse SDK] 401: Unauthorized. Please check your public/private host settings. Refer to our installation and setup guide: https://langfuse.com/docs/sdk/typescript/guide for details on SDK configuration. Make sure to keep your SDK updated, refer to https://www.npmjs.com/package/langfuse for details. Error details: LangfuseFetchHttpError: HTTP error while fetching Langfuse: 401 and body: {"message":"Invalid authorization header"}
    1142:  TypeError: itemsResponse.data is not iterable (cannot read property undefined)
    1143:  at Langfuse.getDataset (/home/runner/work/liam/liam/node_modules/.pnpm/langfuse-core@3.37.2/node_modules/langfuse-core/src/index.ts:1432:13)
    ...
    
    1151:  �[1m�[46m RUN �[49m�[22m �[36mv3.1.2 �[39m�[90m/home/runner/work/liam/liam/frontend/packages/db-structure�[39m
    1152:  �[32m✓�[39m src/parser/tbls/index.test.ts �[2m(�[22m�[2m20 tests�[22m�[2m)�[22m�[32m 49�[2mms�[22m�[39m
    1153:  (node:7713) ExperimentalWarning: WASI is an experimental feature and might change at any time
    1154:  (Use `node --trace-warnings ...` to show where the warning was created)
    1155:  �[32m✓�[39m src/parser/prisma/index.test.ts �[2m(�[22m�[2m21 tests�[22m�[2m)�[22m�[33m 433�[2mms�[22m�[39m
    1156:  �[32m✓�[39m src/parser/schemarb/index.test.ts �[2m(�[22m�[2m18 tests�[22m�[2m)�[22m�[33m 560�[2mms�[22m�[39m
    1157:  �[32m✓�[39m src/schema/overrideSchema.test.ts �[2m(�[22m�[2m6 tests�[22m�[2m)�[22m�[32m 10�[2mms�[22m�[39m
    1158:  �[32m✓�[39m src/parser/sql/postgresql/processSQLInChunks.test.ts �[2m(�[22m�[2m6 tests�[22m�[2m)�[22m�[32m 37�[2mms�[22m�[39m
    1159:  �[32m✓�[39m src/parser/tbls/schema.generated.test.ts �[2m(�[22m�[2m5 tests�[22m�[2m)�[22m�[32m 50�[2mms�[22m�[39m
    1160:  �[32m✓�[39m src/parser/schemarb/singularize.test.ts �[2m(�[22m�[2m18 tests�[22m�[2m)�[22m�[32m 27�[2mms�[22m�[39m
    1161:  �[32m✓�[39m src/parser/supportedFormat/detectFormat.test.ts �[2m(�[22m�[2m10 tests�[22m�[2m)�[22m�[32m 13�[2mms�[22m�[39m
    1162:  (node:8047) ExperimentalWarning: WASI is an experimental feature and might change at any time
    1163:  (Use `node --trace-warnings ...` to show where the warning was created)
    1164:  �[32m✓�[39m src/parser/index.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m)�[22m�[32m 285�[2mms�[22m�[39m
    1165:  �[32m✓�[39m src/parser/sql/postgresql/index.test.ts �[2m(�[22m�[2m19 tests�[22m�[2m | �[22m�[33m1 skipped�[39m�[2m)�[22m�[33m 43361�[2mms�[22m�[39m
    1166:  �[33m�[2m✓�[22m�[39m processor�[2m > �[22mabnormal cases�[2m > �[22mshow error if the syntax is broken �[33m 307�[2mms�[22m�[39m
    1167:  �[33m�[2m✓�[22m�[39m processor�[2m > �[22mLong "create table" statement (exceeds 500 lines, surpassing CHUNK_SIZE)�[2m > �[22mparses without errors �[33m 20539�[2mms�[22m�[39m
    1168:  �[33m�[2m✓�[22m�[39m processor�[2m > �[22mLong "create function" statement (exceeds 500 lines, surpassing CHUNK_SIZE)�[2m > �[22mparses without errors �[33m 21732�[2mms�[22m�[39m
    1169:  �[2m Test Files �[22m �[1m�[32m10 passed�[39m�[22m�[90m (10)�[39m
    ...
    
    1200:  [vite]  * [new branch]          add-playwright           -> origin/add-playwright
    1201:  [vite]  * [new branch]          add-schema-meta          -> origin/add-schema-meta
    1202:  [vite]  * [new branch]          add_script_to_gen_schema -> origin/add_script_to_gen_schema
    1203:  [vite]  * [new branch]          add_script_to_gen_schema_json -> origin/add_script_to_gen_schema_json
    1204:  [vite]  * [new branch]          add_test_table           -> origin/add_test_table
    1205:  [vite]  * [new branch]          avoid-cp-for-windows-actions -> origin/avoid-cp-for-windows-actions
    1206:  [vite]  * [new branch]          avoid-semicolon-logic-2  -> origin/avoid-semicolon-logic-2
    1207:  [vite]  * [new branch]          avoid-semicolon-logic-test -> origin/avoid-semicolon-logic-test
    1208:  [vite]  * [new branch]          before-promptfoo-experiment -> origin/before-promptfoo-experiment
    1209:  [vite]  * [new branch]          changeset-invest1        -> origin/changeset-invest1
    1210:  [vite]  * [new branch]          changeset-invest3        -> origin/changeset-invest3
    1211:  [vite]  * [new branch]          changeset-release/changeset-invest1 -> origin/changeset-release/changeset-invest1
    1212:  [vite]  * [new branch]          check-github-action      -> origin/check-github-action
    1213:  [vite]  * [new branch]          chore/install-renovate   -> origin/chore/install-renovate
    1214:  [vite]  * [new branch]          cli-schema-json-for-local -> origin/cli-schema-json-for-local
    1215:  [vite]  * [new branch]          console-error-1-draft    -> origin/console-error-1-draft
    1216:  [vite]  * [new branch]          delete-docs-vercel-deploy-erd-yml -> origin/delete-docs-vercel-deploy-erd-yml
    1217:  [vite]  * [new branch]          devin/1738820244-add-init-command-tests -> origin/devin/1738820244-add-init-command-tests
    1218:  [vite]  * [new branch]          devin/1738828298-add-multiline-comment-support -> origin/devin/1738828298-add-multiline-comment-support
    1219:  [vite]  * [new branch]          devin/1739501885-improve-navigation-test-waits -> origin/devin/1739501885-improve-navigation-test-waits
    1220:  [vite]  * [new branch]          devin/1740113652-update-changeset-docs -> origin/devin/1740113652-update-changeset-docs
    1221:  [vite]  * [new branch]          devin/1741661851-optimize-github-config-validation -> origin/devin/1741661851-optimize-github-config-validation
    1222:  [vite]  * [new branch]          devin/1741663537-github-env-validation-at-boot -> origin/devin/1741663537-github-env-validation-at-boot
    1223:  [vite]  * [new branch]          devin/1741825670-postgresql-parser-test -> origin/devin/1741825670-postgresql-parser-test
    1224:  [vite]  * [new branch]          devin/1743062004-investigate-issue-1005 -> origin/devin/1743062004-investigate-issue-1005
    1225:  [vite]  * [new branch]          devin/1744202861-add-organization-ui -> origin/devin/1744202861-add-organization-ui
    1226:  [vite]  * [new branch]          devin/1745381489-fix-supabase-auth-security -> origin/devin/1745381489-fix-supabase-auth-security
    1227:  [vite]  * [new branch]          devin/1745404272-fix-appbar-hydration-error -> origin/devin/1745404272-fix-appbar-hydration-error
    1228:  [vite]  * [new branch]          devin/1745412436-rename-github-doc-file-paths -> origin/devin/1745412436-rename-github-doc-file-paths
    ...
    
    1231:  [vite]  * [new branch]          devin/1745494679-add-organization-id-github-pull-requests -> origin/devin/1745494679-add-organization-id-github-pull-requests
    1232:  [vite]  * [new branch]          devin/1745494685-add-organization-id-to-migrations -> origin/devin/1745494685-add-organization-id-to-migrations
    1233:  [vite]  * [new branch]          devin/1745494737-add-organization-id-schema-file-paths -> origin/devin/1745494737-add-organization-id-schema-file-paths
    1234:  [vite]  * [new branch]          devin/1745497726-decouple-overall-reviews-from-github -> origin/devin/1745497726-decouple-overall-reviews-from-github
    1235:  [vite]  * [new branch]          devin/1745498739-add-organization-id-migration-pull-request-mappings -> origin/devin/1745498739-add-organization-id-migration-pull-request-mappings
    1236:  [vite]  * [new branch]          devin/1745498835-add-organization-id -> origin/devin/1745498835-add-organization-id
    1237:  [vite]  * [new branch]          devin/1745498969-add-organization-id-mapping -> origin/devin/1745498969-add-organization-id-mapping
    1238:  [vite]  * [new branch]          devin/1745499053-add-organization-id-review-feedback-knowledge-suggestion-mappings -> origin/devin/1745499053-add-organization-id-review-feedback-knowledge-suggestion-mappings
    1239:  [vite]  * [new branch]          devin/1745548711-remove-try-catch -> origin/devin/1745548711-remove-try-catch
    1240:  [vite]  * [new branch]          devin/1745550064-remove-jobs-knip-ignores -> origin/devin/1745550064-remove-jobs-knip-ignores
    1241:  [vite]  * [new branch]          emu-pr-1033-base         -> origin/emu-pr-1033-base
    1242:  [vite]  * [new branch]          emu-pr-1033-feature      -> origin/emu-pr-1033-feature
    1243:  [vite]  * [new branch]          emu-pr-1081-base         -> origin/emu-pr-1081-base
    1244:  [vite]  * [new branch]          emu-pr-1081-feature      -> origin/emu-pr-1081-feature
    1245:  [vite]  * [new branch]          enhance-the-highlighting-of-related-table-nodes-when-hovering-usememo-1 -> origin/enhance-the-highlighting-of-related-table-nodes-when-hovering-usememo-1
    1246:  [vite]  * [new branch]          error-message-erd-network-error -> origin/error-message-erd-network-error
    1247:  [vite]  * [new branch]          feat-related-posts       -> origin/feat-related-posts
    ...
    
    1280:  [vite]  * [new branch]          prompt-test-package      -> origin/prompt-test-package
    1281:  [vite]  * [new branch]          promptfoo-go-1           -> origin/promptfoo-go-1
    1282:  [vite]  * [new branch]          reduce-set-node-calling-chance-3 -> origin/reduce-set-node-calling-chance-3
    1283:  [vite]  * [new branch]          refactor-installation-selector -> origin/refactor-installation-selector
    1284:  [vite]  * [new branch]          refactor-to-testcases    -> origin/refactor-to-testcases
    1285:  [vite]  * [new branch]          refactor_edge_cardinality_by_using_custom_mark -> origin/refactor_edge_cardinality_by_using_custom_mark
    1286:  [vite]  * [new branch]          refactor_hidden_node_handle -> origin/refactor_hidden_node_handle
    1287:  [vite]  * [new branch]          release-debug-branch     -> origin/release-debug-branch
    1288:  [vite]  * [new branch]          ren-a-table              -> origin/ren-a-table
    1289:  [vite]  * [new branch]          renovate-pnpm-10.x-2     -> origin/renovate-pnpm-10.x-2
    1290:  [vite]  * [new branch]          renovate/major-react-monorepo -> origin/renovate/major-react-monorepo
    1291:  [vite]  * [new branch]          renovate/node-22.x       -> origin/renovate/node-22.x
    1292:  [vite]  * [new branch]          renovate/playwright-monorepo -> origin/renovate/playwright-monorepo
    1293:  [vite]  * [new branch]          renovate/pnpm-10.x       -> origin/renovate/pnpm-10.x
    1294:  [vite]  * [new branch]          renovate/sentry-javascript-monorepo -> origin/renovate/sentry-javascript-monorepo
    1295:  [vite]  * [new branch]          revert-435-error-message-erd -> origin/revert-435-error-message-erd
    1296:  [vite]  * [new branch]          review-suggestion-patch-suggestion-2 -> origin/review-suggestion-patch-suggestion-2
    ...
    
    1433:  * [new branch]        add-playwright           -> origin/add-playwright
    1434:  * [new branch]        add-schema-meta          -> origin/add-schema-meta
    1435:  * [new branch]        add_script_to_gen_schema -> origin/add_script_to_gen_schema
    1436:  * [new branch]        add_script_to_gen_schema_json -> origin/add_script_to_gen_schema_json
    1437:  * [new branch]        add_test_table           -> origin/add_test_table
    1438:  * [new branch]        avoid-cp-for-windows-actions -> origin/avoid-cp-for-windows-actions
    1439:  * [new branch]        avoid-semicolon-logic-2  -> origin/avoid-semicolon-logic-2
    1440:  * [new branch]        avoid-semicolon-logic-test -> origin/avoid-semicolon-logic-test
    1441:  * [new branch]        before-promptfoo-experiment -> origin/before-promptfoo-experiment
    1442:  * [new branch]        changeset-invest1        -> origin/changeset-invest1
    1443:  * [new branch]        changeset-invest3        -> origin/changeset-invest3
    1444:  * [new branch]        changeset-release/changeset-invest1 -> origin/changeset-release/changeset-invest1
    1445:  * [new branch]        check-github-action      -> origin/check-github-action
    1446:  * [new branch]        chore/install-renovate   -> origin/chore/install-renovate
    1447:  * [new branch]        cli-schema-json-for-local -> origin/cli-schema-json-for-local
    1448:  * [new branch]        console-error-1-draft    -> origin/console-error-1-draft
    1449:  * [new branch]        delete-docs-vercel-deploy-erd-yml -> origin/delete-docs-vercel-deploy-erd-yml
    1450:  * [new branch]        devin/1738820244-add-init-command-tests -> origin/devin/1738820244-add-init-command-tests
    1451:  * [new branch]        devin/1738828298-add-multiline-comment-support -> origin/devin/1738828298-add-multiline-comment-support
    1452:  * [new branch]        devin/1739501885-improve-navigation-test-waits -> origin/devin/1739501885-improve-navigation-test-waits
    1453:  * [new branch]        devin/1740113652-update-changeset-docs -> origin/devin/1740113652-update-changeset-docs
    1454:  * [new branch]        devin/1741661851-optimize-github-config-validation -> origin/devin/1741661851-optimize-github-config-validation
    1455:  * [new branch]        devin/1741663537-github-env-validation-at-boot -> origin/devin/1741663537-github-env-validation-at-boot
    1456:  * [new branch]        devin/1741825670-postgresql-parser-test -> origin/devin/1741825670-postgresql-parser-test
    1457:  * [new branch]        devin/1743062004-investigate-issue-1005 -> origin/devin/1743062004-investigate-issue-1005
    1458:  * [new branch]        devin/1744202861-add-organization-ui -> origin/devin/1744202861-add-organization-ui
    1459:  * [new branch]        devin/1745381489-fix-supabase-auth-security -> origin/devin/1745381489-fix-supabase-auth-security
    1460:  * [new branch]        devin/1745404272-fix-appbar-hydration-error -> origin/devin/1745404272-fix-appbar-hydration-error
    1461:  * [new branch]        devin/1745412436-rename-github-doc-file-paths -> origin/devin/1745412436-rename-github-doc-file-paths
    ...
    
    1464:  * [new branch]        devin/1745494679-add-organization-id-github-pull-requests -> origin/devin/1745494679-add-organization-id-github-pull-requests
    1465:  * [new branch]        devin/1745494685-add-organization-id-to-migrations -> origin/devin/1745494685-add-organization-id-to-migrations
    1466:  * [new branch]        devin/1745494737-add-organization-id-schema-file-paths -> origin/devin/1745494737-add-organization-id-schema-file-paths
    1467:  * [new branch]        devin/1745497726-decouple-overall-reviews-from-github -> origin/devin/1745497726-decouple-overall-reviews-from-github
    1468:  * [new branch]        devin/1745498739-add-organization-id-migration-pull-request-mappings -> origin/devin/1745498739-add-organization-id-migration-pull-request-mappings
    1469:  * [new branch]        devin/1745498835-add-organization-id -> origin/devin/1745498835-add-organization-id
    1470:  * [new branch]        devin/1745498969-add-organization-id-mapping -> origin/devin/1745498969-add-organization-id-mapping
    1471:  * [new branch]        devin/1745499053-add-organization-id-review-feedback-knowledge-suggestion-mappings -> origin/devin/1745499053-add-organization-id-review-feedback-knowledge-suggestion-mappings
    1472:  * [new branch]        devin/1745548711-remove-try-catch -> origin/devin/1745548711-remove-try-catch
    1473:  * [new branch]        devin/1745550064-remove-jobs-knip-ignores -> origin/devin/1745550064-remove-jobs-knip-ignores
    1474:  * [new branch]        emu-pr-1033-base         -> origin/emu-pr-1033-base
    1475:  * [new branch]        emu-pr-1033-feature      -> origin/emu-pr-1033-feature
    1476:  * [new branch]        emu-pr-1081-base         -> origin/emu-pr-1081-base
    1477:  * [new branch]        emu-pr-1081-feature      -> origin/emu-pr-1081-feature
    1478:  * [new branch]        enhance-the-highlighting-of-related-table-nodes-when-hovering-usememo-1 -> origin/enhance-the-highlighting-of-related-table-nodes-when-hovering-usememo-1
    1479:  * [new branch]        error-message-erd-network-error -> origin/error-message-erd-network-error
    1480:  * [new branch]        feat-related-posts       -> origin/feat-related-posts
    ...
    
    1513:  * [new branch]        prompt-test-package      -> origin/prompt-test-package
    1514:  * [new branch]        promptfoo-go-1           -> origin/promptfoo-go-1
    1515:  * [new branch]        reduce-set-node-calling-chance-3 -> origin/reduce-set-node-calling-chance-3
    1516:  * [new branch]        refactor-installation-selector -> origin/refactor-installation-selector
    1517:  * [new branch]        refactor-to-testcases    -> origin/refactor-to-testcases
    1518:  * [new branch]        refactor_edge_cardinality_by_using_custom_mark -> origin/refactor_edge_cardinality_by_using_custom_mark
    1519:  * [new branch]        refactor_hidden_node_handle -> origin/refactor_hidden_node_handle
    1520:  * [new branch]        release-debug-branch     -> origin/release-debug-branch
    1521:  * [new branch]        ren-a-table              -> origin/ren-a-table
    1522:  * [new branch]        renovate-pnpm-10.x-2     -> origin/renovate-pnpm-10.x-2
    1523:  * [new branch]        renovate/major-react-monorepo -> origin/renovate/major-react-monorepo
    1524:  * [new branch]        renovate/node-22.x       -> origin/renovate/node-22.x
    1525:  * [new branch]        renovate/playwright-monorepo -> origin/renovate/playwright-monorepo
    1526:  * [new branch]        renovate/pnpm-10.x       -> origin/renovate/pnpm-10.x
    1527:  * [new branch]        renovate/sentry-javascript-monorepo -> origin/renovate/sentry-javascript-monorepo
    1528:  * [new branch]        revert-435-error-message-erd -> origin/revert-435-error-message-erd
    1529:  * [new branch]        review-suggestion-patch-suggestion-2 -> origin/review-suggestion-patch-suggestion-2
    ...
    
    1626:  * [new tag]           @liam-hq/ui@0.0.15       -> @liam-hq/ui@0.0.15
    1627:  * [new tag]           @liam-hq/ui@0.0.16       -> @liam-hq/ui@0.0.16
    1628:  �[1m�[46m RUN �[49m�[22m �[36mv3.1.2 �[39m�[90m/home/runner/work/liam/liam/frontend/packages/cli�[39m
    1629:  �[32m✓�[39m src/cli/erdCommand/getInputContent.test.ts �[2m(�[22m�[2m8 tests�[22m�[2m)�[22m�[32m 128�[2mms�[22m�[39m
    1630:  (node:9074) ExperimentalWarning: WASI is an experimental feature and might change at any time
    1631:  (Use `node --trace-warnings ...` to show where the warning was created)
    1632:  (node:9162) ExperimentalWarning: WASI is an experimental feature and might change at any time
    1633:  (Use `node --trace-warnings ...` to show where the warning was created)
    1634:  �[32m✓�[39m src/cli/erdCommand/runPreprocess.test.ts �[2m(�[22m�[2m4 tests�[22m�[2m)�[22m�[33m 606�[2mms�[22m�[39m
    1635:  �[33m�[2m✓�[22m�[39m runPreprocess�[2m > �[22mshould create schema.json with the content in 'postgres' format �[33m 364�[2mms�[22m�[39m
    1636:  �[32m✓�[39m src/cli/index.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[32m 20�[2mms�[22m�[39m
    1637:  (node:9329) ExperimentalWarning: WASI is an experimental feature and might change at any time
    1638:  (Use `node --trace-warnings ...` to show where the warning was created)
    1639:  �[32m✓�[39m src/cli/erdCommand/index.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[32m 10�[2mms�[22m�[39m
    1640:  �[32m✓�[39m src/cli/smoke.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m)�[22m�[33m 40881�[2mms�[22m�[39m
    1641:  �[33m�[2m✓�[22m�[39m CLI Smoke Test�[2m > �[22mshould run the CLI command without errors: `erd` �[33m 3063�[2mms�[22m�[39m
    1642:  �[33m�[2m✓�[22m�[39m CLI Smoke Test�[2m > �[22mshould run the CLI command without errors: `erd build` �[33m 4527�[2mms�[22m�[39m
    1643:  �[2m Test Files �[22m �[1m�[32m5 passed�[39m�[22m�[90m (5)�[39m
    ...
    
    1749:  {
    1750:  -            foreignKeyName: 'overall_review_knowledge_suggestion_ma_knowledge_suggestion_id_fkey'
    1751:  +            foreignKeyName: 'overall_review_knowledge_suggestion_mapping_knowledge_suggestio'
    1752:  columns: ['knowledge_suggestion_id']
    1753:  isOneToOne: false
    1754:  referencedRelation: 'knowledge_suggestions'
    1755:  referencedColumns: ['id']
    1756:  },
    1757:  {
    1758:  -            foreignKeyName: 'overall_review_knowledge_suggestion_ma_overall_review_id_fkey'
    1759:  +            foreignKeyName: 'overall_review_knowledge_suggestion_mapping_overall_review_id_f'
    1760:  columns: ['overall_review_id']
    1761:  isOneToOne: false
    1762:  referencedRelation: 'overall_reviews'
    1763:  Generated types and schema.sql differ from committed files.
    1764:  ##[error]Process completed with exit code 1.
    1765:  Post job cleanup.
    

    Copy link
    Contributor

    liam-migration bot commented Apr 17, 2025

    This migration refactors numeric identifiers to strings and updates naming conventions per project rules. The most pressing issue is ensuring atomic rollback and integrity during the type conversion while also addressing potential performance and scalability impacts. Overall, the adherence to naming consistency is a positive step, but clarifications and robust testing for data integrity and performance are essential.

    Migration URL: https://liam-erd-web.vercel.app/app/projects/6/ref/staging/migrations/191

    ER Diagram:

    Copy link

    This migration refactors numeric identifiers to strings and renames columns and tables to enforce project naming conventions. The most critical issue is ensuring that the type conversions and foreign-key relationships are updated atomically to avoid data integrity problems. Overall, the consistent application of project rules and improved schema naming are positive, but clarifications on rollback, performance benchmarking, and data consistency are needed.

    Migration URL: https://liam-app-git-staging-route-06-core.vercel.app/app/projects/9d777f64-400a-42f3-a60e-98a59fc97279/ref/staging/migrations/4016d8d9-5992-40e3-9d07-c01837d2abb4

    ER Diagram:

    devin-ai-integration bot and others added 4 commits April 24, 2025 12:33
    …l_request_id and project_id
    
    Co-Authored-By: noritaka.ikeda@route06.co.jp <noritaka.ikeda@route06.co.jp>
    Co-Authored-By: noritaka.ikeda@route06.co.jp <noritaka.ikeda@route06.co.jp>
    …ect_id
    
    Co-Authored-By: noritaka.ikeda@route06.co.jp <noritaka.ikeda@route06.co.jp>
    …id and pull_request_id
    
    Co-Authored-By: noritaka.ikeda@route06.co.jp <noritaka.ikeda@route06.co.jp>
    Copy link
    Contributor

    liam-migration bot commented Apr 25, 2025

    This migration refactors overall review associations by replacing pull_request_id with migration_id and updating related foreign key mappings in both SQL and application code. The most critical concern is the potential data loss and performance issues that may arise from the DELETE and UPDATE operations within the transaction. The changes adhere well to project naming conventions and decouple GitHub dependencies, though clarifications on backup and rollback strategies are warranted.

    Migration URL: https://liam-erd-web.vercel.app/app/projects/284f1bca-2633-4a93-9567-7bc98ab298f7/ref/staging/migrations/e2a2c531-297a-45d4-a18b-88a1e2d72527

    ER Diagram:

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    None yet
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants