feat(agent-task): add Qovery service context - #2966
Conversation
Allow users to select services from the current environment and append their identifiers to the submitted agent prompt.
|
View your CI Pipeline Execution ↗ for commit 28df3ee
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review completed against the latest diff
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 6 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## staging #2966 +/- ##
===========================================
+ Coverage 46.62% 50.67% +4.04%
===========================================
Files 517 1338 +821
Lines 13628 28790 +15162
Branches 4221 8462 +4241
===========================================
+ Hits 6354 14588 +8234
- Misses 6203 11791 +5588
- Partials 1071 2411 +1340
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
1 issue found across 3 files (changes from recent commits).
Confidence score: 4/5
- In
qovery-service-context-modal.tsx, selecting every available service removesSelect allwithout offering a clear-all action, forcing users to uncheck items individually; retain aReset allorClear allcontrol for the editing flow.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-configuration/context/qovery-service-context-modal.tsx">
<violation number="1" location="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-configuration/context/qovery-service-context-modal.tsx:73">
P2: When every available service is selected, this modal now hides `Select all` and provides no clear-all action, so users must uncheck services one by one. Keep a `Reset all` or `Clear all` action for the editing flow.</violation>
</file>
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 17 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 28 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 36 files (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 36 files (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 36 files (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
1 issue found across 37 files (changes from recent commits).
Confidence score: 3/5
libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-request.tscan leave an edited service entry alongside a newly generated block when processing a legacy unmarked context, producing duplicate or incorrect workflow output; adjust replacement and edited-block detection ordering to preserve the edited entry correctly.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-request.ts">
<violation number="1" location="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-request.ts:37">
P2: When a legacy unmarked context block has an edited later service entry, this replacement strips only the generated prefix before edited-block detection runs. The edited entry then survives beside the newly generated block; replace the whole legacy section atomically or detect edited entries before removing its heading.</violation>
</file>
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
1 existing issue remains and no new issues found across 33 files (changes from recent commits).
Confidence score: 2/5
- In
libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-request.ts,replaceContextServicesInPromptcan consume and drop every subsequent nonblank line after an edited legacy context section, truncating the agent prompt and likely breaking workflow behavior—fix the matching logic and add coverage for a nonblank instruction following the edited section.
Requires human review: Auto-approval blocked because this review re-detected 2 unresolved issues already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 33 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
| <Navigate | ||
| to="/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/agentic-workflow/configuration" | ||
| params={{ organizationId, projectId, environmentId }} | ||
| search={(search) => search} |
There was a problem hiding this comment.
Necessary to keep template URL
There was a problem hiding this comment.
1 issue found across 33 files (changes from recent commits).
Confidence score: 5/5
- In
git-context-card.tsx, configured-provider badges may show raw enum values such asGITHUBinstead of user-friendly names likeGitHub, creating a minor presentation regression; restore or reuse thePROVIDER_LABELSmapping.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-configuration/context/git-context-card.tsx">
<violation number="1" location="libs/domains/services/feature/src/lib/service-creation-flow/agentic-workflow/agentic-workflow-configuration/context/git-context-card.tsx:35">
P3: The badge label for a configured provider now renders the raw enum code ('GITHUB', 'GITLAB', 'BITBUCKET') instead of the previous friendly names ('GitHub', 'GitLab', 'Bitbucket'), because the PROVIDER_LABELS mapping was replaced with `provider.toUpperCase()`. If the uppercase mono look is intended, fine — but if the change was meant only to drop the mapping, keep the friendly labels and confirm the new casing matches the design.</violation>
</file>
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
| requiresQoveryMcp={selectedTemplate?.requiresQoveryMcp} | ||
| seed={selectedTemplate?.seed} | ||
| variablesSeed={selectedTemplate?.variables} |
There was a problem hiding this comment.
Maybe we could pass the selectedTemplate altogether instead of having multiple props
|
🎉 This PR is included in version 1.354.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
2 participants
Summary
Context servicesin the submitted agent prompt.Screenshots / Recordings
Testing
yarn test(31 focused tests)yarn prettier --checkyarn eslintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-releaseSummary by cubic
Agent Tasks previously could not use Qovery services as context or enforce template MCP requirements. They now support both while preserving valid context and required MCP selections during configuration.
Details
Add contextmulti-select modal for services in the current environment, excluding other Agent Tasks.context_service_idsand replaces the delimited generatedContext servicesprompt block with their names, types, and IDs.qovery-typescript-axiosto1.1.976.Written for commit 40f4f7f. Summary will update on new commits.