refactor: rename node type taxonomy (Data* → Job*/Process*/Limit*/Billing*) + add ProcessExtraction, ProcessImageVersions#1164
Merged
Conversation
…cessExtraction, ProcessImageVersions Renames: - DataJobTrigger → JobTrigger (category: job) - DataFileEmbedding → ProcessFileEmbedding (category: process) - DataChunks → ProcessChunks (category: process) - DataImageEmbedding → ProcessImageEmbedding (category: process) New nodes: - ProcessExtraction: file text extraction pipeline (output fields + MIME-conditional job trigger) - ProcessImageVersions: image variant generation pipeline (MIME-conditional job trigger) Also adds 'check', 'job', 'process' to codegen categoryOrder so all categories generate proper Params interfaces. Refs: constructive-planning#857, constructive-planning#848
Contributor
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…Counter, DataAggregateLimitCounter → LimitAggregate, DataFeatureFlag → LimitFeatureFlag, DataBillingMeter → BillingMeter) (#858)
devin-ai-integration
Bot
changed the title
…ed defaults - versions is now required (minItems: 1) with no default array - Blueprint must explicitly specify what image variants to generate - Prevents opinionated defaults (thumb 150x150, preview 800x600) from silently applying
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Renames node types to use semantic category prefixes that describe what each node creates, rather than the generic
Data*bucket. Also adds two new Phase 2 processing pipeline nodes.Refs: constructive-io/constructive-planning#857, constructive-io/constructive-planning#858
8 renames across 3 new categories:
DataJobTriggerJobTriggerjobDataFileEmbeddingProcessFileEmbeddingprocessDataChunksProcessChunksprocessDataImageEmbeddingProcessImageEmbeddingprocessDataLimitCounterLimitCounterlimitDataAggregateLimitCounterLimitAggregatelimitDataFeatureFlagLimitFeatureFlaglimitDataBillingMeterBillingMeterbilling2 new nodes:
ProcessExtraction(text/metadata extraction),ProcessImageVersions(image resize/crop)Full taxonomy:
Data*(schema),Search*(indexes),Authz*(RLS),Check*(constraints),Limit*(usage enforcement),Billing*(metering),Job*(async work),Process*(pipelines),Relation*(junctions),View*(views)Review & Testing Checklist for Human
Risk: yellow — name changes only, no logic changes. SQL slugs unchanged.
Data*names for the 8 renamed nodescategoryOrderincludes all 10 categories — codegen generates interfaces for billing, limit, job, process categoriespnpm buildinpackages/node-type-registryNotes
data_limit_counter,data_feature_flag, etc.) are unchanged — only the node typenameandcategoryfields changedLink to Devin session: https://app.devin.ai/sessions/2b5a29d83d3f478e8d3d972653b4879c
Requested by: @pyramation