You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
The recent database schema changes introduce a new model, KnowledgeSuggestion, which facilitates the storage and management of AI-generated suggestions for updates to schemas and documentation. This addition is a strategic enhancement to the existing database structure and aligns well with the project's goals of automating review processes and improving efficiency in handling schema changes.
Model Definition: The KnowledgeSuggestion model has been defined with appropriate fields such as id, type, title, path, content, fileSha, projectId, overallReviewId, approvedAt, createdAt, and updatedAt. The use of an enum KnowledgeType allows for clear categorization of suggestions into two types: SCHEMA and DOCS. This is a commendable design choice as it enhances data integrity and allows for easier querying based on type.
Relationships: The model includes foreign key relationships with the Project and OverallReview models, which ensures the integrity of relationships and allows for efficient joins in queries. The nullable overallReviewId field is a sensible design decision, allowing for suggestions that may not yet be linked to a review.
Database Migration: The migration scripts are well-structured, creating the necessary table and constraints for the new model. They include appropriate foreign key constraints that enforce referential integrity. The inclusion of timestamps for creation and updates (createdAt and updatedAt) adheres to best practices for tracking record history.
Frontend Integration: The corresponding additions to the frontend codebase demonstrate a thoughtful integration of the KnowledgeSuggestion model. The new pages for displaying and managing knowledge suggestions (KnowledgeSuggestionsListPage and KnowledgeSuggestionDetailPage) provide a user-friendly interface for users to view and act on suggestions. The use of form validation with valibot ensures that data integrity is maintained when submitting suggestions.
Approving Suggestions: The functionality to approve a knowledge suggestion and update the corresponding file in GitHub is a key feature. The careful handling of form data and the synchronous flow for updating the suggestion status exhibit a solid understanding of asynchronous operations and error handling.
Code Quality: The code introduced is clean and adheres to modern JavaScript/TypeScript standards. The use of async/await patterns for asynchronous operations enhances readability and maintainability. The error handling is robust, with appropriate logging mechanisms in place to assist in troubleshooting.
Testing and Validation: While the schema and code changes are well-structured, it would be prudent to implement unit tests to validate the new functionalities. This would help ensure that the new model behaves as expected and that edge cases are handled gracefully.
Documentation: The documentation updates in progress.md are a good practice, providing context for the new features and their intended functionality. Continual documentation updates will be essential as the project evolves to keep all stakeholders informed.
Potential Enhancements: Future enhancements could include additional fields in the KnowledgeSuggestion model, such as user feedback or categories for suggestions to further refine the suggestion process. Implementing a notification system for users when a suggestion is approved could also improve engagement.
Overall, the schema changes are a significant and well-executed enhancement to the database structure, facilitating improved management of knowledge suggestions in the context of schema and documentation updates. The changes align well with the overall project goals and demonstrate a solid architectural design. Further testing and documentation will bolster the robustness of this implementation as it moves forward.
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
✅
Thu, 03 Apr 2025 06:25:09 UTC
Migrations
❌
Thu, 03 Apr 2025 06:25:12 UTC
Seeding
⏸️
Thu, 03 Apr 2025 06:24:59 UTC
Edge Functions
⏸️
Thu, 03 Apr 2025 06:24:59 UTC
❌ Branch Error • Thu, 03 Apr 2025 06:25:13 UTC
Remote migration versions not found in local migrations directory.
Failed test name: postComment > should create a new comment when no comment exists
Failure summary:
The action failed because of test failures in the @liam-hq/jobs package. Specifically:
1. Two tests in the postComment.test.ts file failed: - should create a new comment when no comment exists - The test expected the function to be called with specific arguments but received different values (PR ID 9999 instead of 1). - should throw error when pull request not found - The test expected an error message about "Pull request with ID 999999 not found" but instead got "Repository with ID 9999 not found".
2. Additionally, there was a failure in the app tests: - getProjectRepository > should return repository information with the correct structure - Expected null not to be null.
Relevant error logs:
1: ##[group]Operating System2: Ubuntu
...
172: �[36;1mpnpm install --frozen-lockfile --prefer-offline�[0m173: shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}174: env:175: PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin176: ##[endgroup]177: Scope: all 16 workspace projects178: Lockfile is up to date, resolution step is skipped179: Progress: resolved 1, reused 0, downloaded 0, added 0180: Packages: +2081181: ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++182: Progress: resolved 2081, reused 949, downloaded 0, added 0183: Progress: resolved 2081, reused 2058, downloaded 0, added 210184: Progress: resolved 2081, reused 2058, downloaded 0, added 843185: Progress: resolved 2081, reused 2058, downloaded 0, added 1635186: Progress: resolved 2081, reused 2058, downloaded 0, added 2081, done187: 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'188: devDependencies:189: + @changesets/cli 2.27.10190: + @changesets/get-github-info 0.6.0191: + @changesets/types 6.0.0192: + @turbo/gen 2.1.2193: + syncpack 13.0.0194: + turbo 2.1.2195: + vercel 41.4.0196: frontend/apps/docs postinstall$ fumadocs-mdx197: frontend/apps/docs postinstall: [MDX] types generated198: frontend/apps/docs postinstall: Done199: frontend/apps/app postinstall$ cp ../../packages/db-structure/node_modules/@ruby/prism/src/prism.wasm prism.wasm200: frontend/apps/app postinstall: Done201: 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'202: Done in 9s
...
930: ##[endgroup]931: ##[group]@liam-hq/erd-core:gen932: cache miss, executing c2780ae6c0cc6d8a933: > @liam-hq/erd-core@0.1.6 gen /home/runner/work/liam/liam/frontend/packages/erd-core934: > pnpm run '/^gen:.*/'935: > @liam-hq/erd-core@0.1.6 gen:css /home/runner/work/liam/liam/frontend/packages/erd-core936: > tcm src937: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/styles/variables.css.d.ts938: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/styles/globals.css.d.ts939: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ERDRenderer.module.css.d.ts940: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDContent/ERDContent.module.css.d.ts941: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/Toolbar/DesktopToolbar.module.css.d.ts942: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/TableDetailDrawer/TableDetailDrawer.module.css.d.ts943: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/RelationshipEdgeParticleMarker/RelationshipEdgeParticleMarker.module.css.d.ts944: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/LeftPane/LeftPane.module.css.d.ts945: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/ParseErrorDisplay.module.css.d.ts946: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/NetworkErrorDisplay.module.css.d.ts947: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/ErrorDisplay/ErrorDisplay.module.css.d.ts948: Wrote /home/runner/work/liam/liam/frontend/packages/erd-core/src/features/erd/components/ERDRenderer/CardinalityMarkers/CardinalityMarkers.module.css.d.ts
...
1024: �[32m✓�[39m src/utils/compressionString.test.ts �[2m(�[22m�[2m5 tests�[22m�[2m)�[22m�[33m 400�[2mms�[22m�[39m1025: �[2m Test Files �[22m �[1m�[32m3 passed�[39m�[22m�[90m (3)�[39m1026: �[2m Tests �[22m �[1m�[32m17 passed�[39m�[22m�[90m (17)�[39m1027: �[2m Start at �[22m 00:57:081028: �[2m Duration �[22m 1.83s�[2m (transform 784ms, setup 0ms, collect 1.16s, tests 500ms, environment 1ms, prepare 1.32s)�[22m1029: ##[endgroup]1030: �[;31m@liam-hq/jobs:test�[;0m1031: cache miss, executing 4320fcf7a46dcc311032: > @liam-hq/jobs@0.1.0 test /home/runner/work/liam/liam/frontend/packages/jobs1033: > vitest --watch=false1034: �[1m�[7m�[36m RUN �[39m�[27m�[22m �[36mv3.0.8 �[39m�[90m/home/runner/work/liam/liam/frontend/packages/jobs�[39m1035: (node:8526) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.1036: (Use `node --trace-deprecation ...` to show where the warning was created)1037: (node:8532) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.1038: (Use `node --trace-deprecation ...` to show where the warning was created)1039: �[90mstderr�[2m | src/functions/__tests__/processSaveReview.test.ts�[2m > �[22m�[2mprocessSaveReview�[2m > �[22m�[2mshould throw error when pull request not found1040: �[22m�[39mError saving review: Error: PullRequest not found: {"code":"PGRST116","details":"The result contains 0 rows","hint":null,"message":"JSON object requested, multiple (or no) rows returned"}1041: at processSaveReview �[90m(/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/processSaveReview.ts:16:13�[90m)�[39m1042: �[90m at processTicksAndRejections (node:internal/process/task_queues:105:5)�[39m1043: at �[90m/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/__tests__/processSaveReview.test.ts:147:51044: at file:///home/runner/work/liam/liam/node_modules/�[4m.pnpm�[24m/@vitest+runner@3.0.8/node_modules/�[4m@vitest�[24m/runner/dist/index.js:563:221045: �[90mstderr�[2m | src/functions/__tests__/postComment.test.ts�[2m > �[22m�[2mpostComment�[2m > �[22m�[2mshould throw error when repository not found1046: �[22m�[39mError posting comment: Error: Repository with ID 999999 not found: JSON object requested, multiple (or no) rows returned1047: at Module.postComment �[90m(/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/postComment.ts:23:13�[90m)�[39m1048: �[90m at processTicksAndRejections (node:internal/process/task_queues:105:5)�[39m1049: at �[90m/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/__tests__/postComment.test.ts:151:51050: at file:///home/runner/work/liam/liam/node_modules/�[4m.pnpm�[24m/@vitest+runner@3.0.8/node_modules/�[4m@vitest�[24m/runner/dist/index.js:563:221051: �[90mstderr�[2m | src/functions/__tests__/processSaveReview.test.ts�[2m > �[22m�[2mprocessSaveReview�[2m > �[22m�[2mshould throw error when creating overall review fails1052: �[22m�[39mError saving review: Error: Failed to create overall review: {"code":"23503","details":"Key (projectId)=(999999) is not present in table \"Project\".","hint":null,"message":"insert or update on table \"OverallReview\" violates foreign key constraint \"OverallReview_projectId_fkey\""}1053: at processSaveReview �[90m(/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/processSaveReview.ts:37:13�[90m)�[39m1054: �[90m at processTicksAndRejections (node:internal/process/task_queues:105:5)�[39m1055: at �[90m/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/__tests__/processSaveReview.test.ts:166:51056: at file:///home/runner/work/liam/liam/node_modules/�[4m.pnpm�[24m/@vitest+runner@3.0.8/node_modules/�[4m@vitest�[24m/runner/dist/index.js:563:221057: �[32m✓�[39m src/functions/__tests__/processSaveReview.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[33m 678�[2mms�[22m�[39m1058: �[90mstderr�[2m | src/functions/__tests__/postComment.test.ts�[2m > �[22m�[2mpostComment�[2m > �[22m�[2mshould throw error when pull request not found1059: �[33m�[2m✓�[22m�[39m processSaveReview�[2m > �[22mshould save review successfully �[33m413�[2mms�[22m�[39m1060: �[22m�[39mError posting comment: Error: Repository with ID 9999 not found: JSON object requested, multiple (or no) rows returned1061: at Module.postComment �[90m(/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/postComment.ts:23:13�[90m)�[39m1062: �[90m at processTicksAndRejections (node:internal/process/task_queues:105:5)�[39m1063: at �[90m/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/__tests__/postComment.test.ts:165:51064: at file:///home/runner/work/liam/liam/node_modules/�[4m.pnpm�[24m/@vitest+runner@3.0.8/node_modules/�[4m@vitest�[24m/runner/dist/index.js:563:221065: �[90mstderr�[2m | src/functions/__tests__/postComment.test.ts�[2m > �[22m�[2mpostComment�[2m > �[22m�[2mshould throw error when migration not found1066: �[22m�[39mError posting comment: Error: Migration for Pull request with ID 8888 not found1067: at Module.postComment �[90m(/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/postComment.ts:47:13�[90m)�[39m1068: �[90m at processTicksAndRejections (node:internal/process/task_queues:105:5)�[39m1069: at �[90m/home/runner/work/liam/liam/frontend/packages/jobs/�[39msrc/functions/__tests__/postComment.test.ts:192:71070: at file:///home/runner/work/liam/liam/node_modules/�[4m.pnpm�[24m/@vitest+runner@3.0.8/node_modules/�[4m@vitest�[24m/runner/dist/index.js:563:221071: �[31m❯�[39m src/functions/__tests__/postComment.test.ts �[2m(�[22m�[2m5 tests�[22m�[2m | �[22m�[31m2 failed�[39m�[2m)�[22m�[33m 817�[2mms�[22m�[39m1072: �[31m �[31m×�[31m postComment�[2m > �[22mshould create a new comment when no comment exists�[33m 450�[2mms�[22m�[31m�[39m
...
1076: �[22m�[2m [�[22m1077: �[2m 12345,�[22m1078: �[2m "test-owner",�[22m1079: �[2m "test-repo",�[22m1080: �[32m- 1,�[90m1081: �[32m- StringContaining "Test review comment",�[90m1082: �[31m+ 9999,�[90m1083: �[31m+ "Test review comment�[90m1084: �[31m+�[90m1085: �[31m+ Migration URL: http://localhost:3000/app/migrations/9999",�[90m1086: �[2m ]�[22m1087: �[31m�[90m1088: Number of calls: �[1m1�[22m1089: �[31m�[39m1090: �[32m✓�[39m postComment�[2m > �[22mshould update existing comment when comment exists1091: �[32m✓�[39m postComment�[2m > �[22mshould throw error when repository not found1092: �[31m �[31m×�[31m postComment�[2m > �[22mshould throw error when pull request not found�[90m 74�[2mms�[22m�[31m�[39m1093: �[31m → expected [Function] to throw error matching /Pull request with ID 999999 not found/ but got 'Repository with ID 9999 not found: JS…'�[39m1094: �[32m✓�[39m postComment�[2m > �[22mshould throw error when migration not found1095: �[31m⎯⎯⎯⎯⎯⎯⎯�[1m�[7m Failed Tests 2 �[27m�[22m⎯⎯⎯⎯⎯⎯⎯�[39m1096: �[31m�[1m�[7m FAIL �[27m�[22m�[39m src/functions/__tests__/postComment.test.ts�[2m > �[22mpostComment�[2m > �[22mshould create a new comment when no comment exists1097: �[31m�[1mAssertionError�[22m: expected "spy" to be called with arguments: [ 12345, 'test-owner', …(3) ]�[90m1098: Received:
...
1107: �[31m+ "Test review comment�[90m1108: �[31m+�[90m1109: �[31m+ Migration URL: http://localhost:3000/app/migrations/9999",�[90m1110: �[2m ]�[22m1111: �[31m�[90m1112: Number of calls: �[1m1�[22m1113: �[31m�[39m1114: �[36m �[2m❯�[22m src/functions/__tests__/postComment.test.ts:�[2m94:38�[22m�[39m1115: �[90m 92| �[39m1116: �[90m 93| �[39m �[34mexpect�[39m(result�[33m.�[39msuccess)�[33m.�[39m�[34mtoBe�[39m(�[35mtrue�[39m)1117: �[90m 94| �[39m �[34mexpect�[39m(createPullRequestComment)�[33m.�[39m�[34mtoHaveBeenCalledWith�[39m(1118: �[90m | �[39m �[31m^�[39m1119: �[90m 95| �[39m testRepository�[33m.�[39minstallationId�[33m,�[39m1120: �[90m 96| �[39m testRepository�[33m.�[39mowner�[33m,�[39m1121: �[31m�[2m⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/2]⎯�[22m�[39m1122: �[31m�[1m�[7m FAIL �[27m�[22m�[39m src/functions/__tests__/postComment.test.ts�[2m > �[22mpostComment�[2m > �[22mshould throw error when pull request not found1123: �[31m�[1mAssertionError�[22m: expected [Function] to throw error matching /Pull request with ID 999999 not found/ but got 'Repository with ID 9999 not found: JS…'�[39m1124: �[32m- Expected:�[39m 1125: /Pull request with ID 999999 not found/1126: �[31m+ Received:�[39m 1127: "Repository with ID 9999 not found: JSON object requested, multiple (or no) rows returned"1128: �[36m �[2m❯�[22m src/functions/__tests__/postComment.test.ts:�[2m165:5�[22m�[39m1129: �[90m163| �[39m }1130: �[90m164| �[39m1131: �[90m165| �[39m �[35mawait�[39m �[34mexpect�[39m(�[34mpostComment�[39m(testPayload))�[33m.�[39mrejects�[33m.�[39m�[34mtoThrow�[39m(1132: �[90m | �[39m �[31m^�[39m1133: �[90m166| �[39m �[36m/Pull request with ID 999999 not found/�[39m�[33m,�[39m1134: �[90m167| �[39m )1135: �[31m�[2m⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[2/2]⎯�[22m�[39m1136: �[2m Test Files �[22m �[1m�[31m1 failed�[39m�[22m�[2m | �[22m�[1m�[32m1 passed�[39m�[22m�[90m (2)�[39m1137: �[2m Tests �[22m �[1m�[31m2 failed�[39m�[22m�[2m | �[22m�[1m�[32m6 passed�[39m�[22m�[90m (8)�[39m1138: �[2m Start at �[22m 00:57:081139: �[2m Duration �[22m 3.05s�[2m (transform 716ms, setup 0ms, collect 2.56s, tests 1.50s, environment 0ms, prepare 792ms)�[22m1140: ELIFECYCLE Test failed. See above for more details.1141: [ERROR] command finished with error: command (/home/runner/work/liam/liam/frontend/packages/jobs) /home/runner/setup-pnpm/node_modules/.bin/pnpm run test exited (1)1142: ##[group]@liam-hq/db-structure:test
...
1158: (Use `node --trace-warnings ...` to show where the warning was created)1159: �[32m✓�[39m src/parser/index.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m)�[22m�[33m 443�[2mms�[22m�[39m1160: ##[endgroup]1161: ##[group]@liam-hq/app:test1162: cache miss, executing d523bf3791c96cf31163: > @liam-hq/app@0.1.0 test /home/runner/work/liam/liam/frontend/apps/app1164: > vitest --watch=false1165: �[33mThe CJS build of Vite's Node API is deprecated. See https://vite.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.�[39m1166: �[1m�[7m�[36m RUN �[39m�[27m�[22m �[36mv3.0.8 �[39m�[90m/home/runner/work/liam/liam/frontend/apps/app�[39m1167: (node:8592) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.1168: (Use `node --trace-deprecation ...` to show where the warning was created)1169: (node:8591) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.1170: (Use `node --trace-deprecation ...` to show where the warning was created)1171: (node:8578) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.1172: (Use `node --trace-deprecation ...` to show where the warning was created)1173: �[31m❯�[39m app/(app)/app/projects/[projectId]/ref/[branchOrCommit]/docs/[...docFilePath]/getProjectRepository.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m | �[22m�[31m1 failed�[39m�[2m)�[22m�[90m 150�[2mms�[22m�[39m1174: �[31m �[31m×�[31m getProjectRepository�[2m > �[22mshould return repository information with the correct structure�[90m 127�[2mms�[22m�[31m�[39m1175: �[31m → expected null not to be null�[39m1176: �[32m✓�[39m getProjectRepository�[2m > �[22mshould return null when project does not exist1177: �[32m✓�[39m app/(app)/app/projects/[projectId]/ref/[branchOrCommit]/docs/getGitHubDocFilePaths.test.ts �[2m(�[22m�[2m2 tests�[22m�[2m)�[22m�[90m 282�[2mms�[22m�[39m1178: �[32m✓�[39m app/api/webhook/github/utils/__tests__/checkSchemaChanges.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[33m 362�[2mms�[22m�[39m1179: ##[endgroup]1180: ##[group]@liam-hq/cli:build1181: cache miss, executing 63d1034a50ae04971182: > @liam-hq/cli@0.5.2 build /home/runner/work/liam/liam/frontend/packages/cli1183: > pnpm run '/^build:.*/'1184: . build:cli$ rollup -c && pnpm run cp:prism1185: . build:vite$ vite build1186: . build:cli: �[36m1187: . build:cli: �[1mbin/cli.ts�[22m → �[1mdist-cli/bin/cli.js�[22m...�[39m1188: . build:vite: Failed1189: . build:cli: Failed1190: ##[endgroup]1191: ##[group]@liam-hq/cli:test1192: cache miss, executing 3a3306233cef7d921193: > @liam-hq/cli@0.5.2 test /home/runner/work/liam/liam/frontend/packages/cli1194: > vitest run1195: Failed during git operations: Error: Command failed: git fetch --tags1196: at genericNodeError (node:internal/errors:983:15)1197: at wrappedFn (node:internal/errors:537:14)1198: at checkExecSyncError (node:child_process:888:11)1199: at execSync (node:child_process:960:15)
...
1205: at async createViteServer (file:///home/runner/work/liam/liam/node_modules/.pnpm/vitest@3.0.8_@edge-runtime+vm@3.2.0_@types+debug@4.1.12_@types+node@22.9.0_jiti@2.4.2_lightni_nle5cbqnt53vrkwed4oeezwtyq/node_modules/vitest/dist/chunks/cli-api.Dis64jtY.js:10300:18) {1206: status: null,1207: signal: 'SIGINT',1208: output: [ null, <Buffer >, <Buffer > ],1209: pid: 8535,1210: stdout: <Buffer >,1211: stderr: <Buffer >1212: }1213: ##[endgroup]1214: ##[group]@liam-hq/prompt-test:test1215: cache miss, executing 6b177102eea47f7e1216: > @liam-hq/prompt-test@0.1.0 test /home/runner/work/liam/liam/frontend/packages/prompt-test1217: > tsx src/index.ts1218: 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.1219: ##[endgroup]1220: ##[error]@liam-hq/jobs#test: command (/home/runner/work/liam/liam/frontend/packages/jobs) /home/runner/setup-pnpm/node_modules/.bin/pnpm run test exited (1)1221: Tasks: 11 successful, 17 total1222: Cached: 0 cached, 17 total1223: Time: 20.014s 1224: Failed: @liam-hq/jobs#test1225: ERROR run failed: command exited (1)1226: ELIFECYCLE Command failed with exit code 1.1227: ##[error]Process completed with exit code 1.1228: Post job cleanup.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
MH4GF
force-pushed
the
staging
branch
2 times, most recently
from
Issue
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