[ENG-1852] Keep Roam shared node content fresh#1147
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
PR size/scope checkThis PR is over our review-size guideline.
Please split this into smaller PRs unless there is a clear reason the changes need to land together. If keeping it as one PR, please add a brief justification covering:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 554884cf58
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
554884c to
116e32c
Compare
Summary
:page/edit-timeor title-level:edit/time.fullmarkdown content without running embeddings, while keeping direct/direct_and_description embedding updates on the existing changed-node path.Base / dependencies
ENG-1848 and ENG-1849 have landed on
main, so this PR has been rebased onto currentmainand now contains only the ENG-1852 freshness-layer commit.Validation
pnpm install --frozen-lockfileafter rebasing onto currentmainpnpm --filter roam check-typespnpm --filter @repo/database check-typespnpm --filter roam lint(0 errors, existing warnings remain)git diff --check origin/main...HEADNote: local shell is still on Node
20.20.1, while currentmainasks for Node>=22, so pnpm prints an engine warning locally.