Skip to content

ENG-1822 Modify upsert_concepts so it accepts embedded content#1097

Open
maparent wants to merge 7 commits into
mainfrom
eng-1822-modify-upsert-concepts-to-accept-embedded-content
Open

ENG-1822 Modify upsert_concepts so it accepts embedded content#1097
maparent wants to merge 7 commits into
mainfrom
eng-1822-modify-upsert-concepts-to-accept-embedded-content

Conversation

@maparent
Copy link
Copy Markdown
Collaborator

@maparent maparent commented May 30, 2026

https://linear.app/discourse-graphs/issue/ENG-1822/modify-upsert-concepts-so-it-accepts-embedded-content

https://www.loom.com/share/cb4e428ed0d647208a78a70fc31b734b

Forgot a point in the loom: The changes to the functions are such that it can still be called exactly as before without changes.


Open in Devin Review

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 30, 2026

ENG-1822

@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app Bot commented May 30, 2026

PR size/scope check

This PR is over our review-size guideline.

  • Recommended: ~200 lines changed
  • Acceptable limit: up to 400 lines when well-scoped/self-contained
  • Preferred file count: fewer than 5 files

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:

  • What single problem this PR solves
  • Why the files/changes are coupled

@supabase
Copy link
Copy Markdown

supabase Bot commented May 30, 2026

Updates to Preview Branch (eng-1822-modify-upsert-concepts-to-accept-embedded-content) ↗︎

Deployments Status Updated
Database Sat, 30 May 2026 14:42:46 UTC
Services Sat, 30 May 2026 14:42:46 UTC
APIs Sat, 30 May 2026 14:42:46 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 Sat, 30 May 2026 14:42:46 UTC
Migrations Sat, 30 May 2026 14:42:46 UTC
Seeding Sat, 30 May 2026 14:42:47 UTC
Edge Functions Sat, 30 May 2026 14:42:50 UTC

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

@maparent
Copy link
Copy Markdown
Collaborator Author

maparent commented May 30, 2026

Note on size: The core of the line count that requires analysis is in concept.sql (81l). This is then duplicated in the migration, propagated to dbTypes. The new types propagate on inputTypes.
addConcept.feature is shuffled to exercise the new code, but it's mostly block moves.
The documentation was made into a compilable ts file; more reorg than changes.
Then small changes to use common types in roam sync and fix discrepancies.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

View 5 additional findings in Devin Review.

Open in Devin Review

Comment thread packages/database/supabase/schemas/concept.sql
@maparent maparent requested a review from mdroidian May 30, 2026 14:58
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.

1 participant