liam icon indicating copy to clipboard operation
liam copied to clipboard

[DO NOT MERGE] test: https://github.com/liam-hq/liam/pull/1081 again

Open hoshinotsuyoshi opened this issue 7 months ago • 4 comments
trafficstars

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

hoshinotsuyoshi avatar Apr 07 '25 12:04 hoshinotsuyoshi

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 7, 2025 0:22am
liam-erd-sample ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 7, 2025 0:22am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
liam-docs ⬜️ Ignored (Inspect) Apr 7, 2025 0:22am

vercel[bot] avatar Apr 07 '25 12:04 vercel[bot]

⚠️ No Changeset found

Latest commit: 65cef3faa8916f81a5ae6dd5266d354090b51ba3

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

changeset-bot[bot] avatar Apr 07 '25 12:04 changeset-bot[bot]

This PR migrates the schema by renaming the WatchSchemaFilePattern table to GitHubSchemaFilePath with data copied and API references updated. The key risk is ensuring data integrity and migration safety through backup and verification procedures to avoid data loss. The change is positive as direct path comparison improves performance and aligns naming conventions for consistency across the project.

Migration URL: https://liam-app-git-staging-route-06-core.vercel.app/app/projects/6/ref/emu-pr-1081-feature/migrations/173

ER Diagram:

  • View ERD for frontend/packages/db/schema/schema.sql: https://liam-app-git-staging-route-06-core.vercel.app/app/projects/6/ref/emu-pr-1081-feature/schema/frontend/packages/db/schema/schema.sql

This PR migrates schema management by replacing the WatchSchemaFilePattern table with GitHubSchemaFilePath and updating all related code and tests. The main risks include insufficient backup measures and lack of post-migration verification which could compromise data integrity and performance monitoring. Overall, the renaming and shift to direct path comparison improve consistency, performance, and security, provided the suggested verifications and documentation updates are implemented.

Migration URL: https://liam-erd-web.vercel.app/app/projects/6/ref/emu-pr-1081-feature/migrations/173

ER Diagram:

  • View ERD for frontend/packages/db/schema/schema.sql: https://liam-erd-web.vercel.app/app/projects/6/ref/emu-pr-1081-feature/schema/frontend/packages/db/schema/schema.sql

liam-migration[bot] avatar Apr 07 '25 12:04 liam-migration[bot]