-
Notifications
You must be signed in to change notification settings - Fork 3.3k
feat(deployments): human-readable version descriptions #3077
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile OverviewGreptile SummaryThis PR enhances deployment version descriptions by resolving IDs to human-readable names. The implementation adds a new Key Changes:
Implementation Quality:
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant DeploymentHook as useGenerateVersionDescription
participant Compare as formatDiffSummaryForDescriptionAsync
participant Resolve as resolveValueForDisplay
participant SelectorRegistry as Selector Registry
participant API as Backend APIs
User->>DeploymentHook: Generate version description
DeploymentHook->>DeploymentHook: Fetch current & previous workflow states
DeploymentHook->>Compare: generateWorkflowDiffSummary(current, previous)
Compare-->>DeploymentHook: WorkflowDiffSummary
DeploymentHook->>Compare: formatDiffSummaryForDescriptionAsync(summary, currentState, workflowId)
loop For each modified block
Compare->>Compare: Extract changes (limit 3 per block)
loop For each change
Compare->>Resolve: resolveValueForDisplay(oldValue, context)
Resolve->>Resolve: Check if resolvable (UUID, credential, etc)
Resolve->>Resolve: Extract extended context from block
alt Is credential field
Resolve->>API: fetchCredentialSetById or fetchOAuthCredentialDetail
API-->>Resolve: Credential name
else Is workflow selector
Resolve->>SelectorRegistry: getSelectorDefinition('sim.workflows')
SelectorRegistry->>API: fetchById(workflowId)
API-->>Resolve: Workflow name
else Is selector type (channel, file, etc)
Resolve->>SelectorRegistry: resolveSelectorForSubBlock
SelectorRegistry->>API: fetchById or fetchList
API-->>Resolve: Display label
end
Resolve-->>Compare: ResolvedValue with displayLabel
Compare->>Resolve: resolveValueForDisplay(newValue, context)
Note over Resolve,API: Same resolution process
Resolve-->>Compare: ResolvedValue with displayLabel
end
end
Compare-->>DeploymentHook: Formatted diff text with human-readable names
DeploymentHook->>API: POST /api/wand (generate AI description)
API-->>DeploymentHook: Streaming description
DeploymentHook-->>User: Final version description
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 files reviewed, no comments
3fef949 to
7658891
Compare
7658891 to
6cb47ac
Compare
6cb47ac to
74c0ba4
Compare
74c0ba4 to
98dfdf9
Compare
98dfdf9 to
1e3ec1f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
1e3ec1f to
088495b
Compare
Summary
Type of Change
Testing
Tested manually
Checklist