feat: implement AI-based deployment job detection in workflows#970
feat: implement AI-based deployment job detection in workflows#970mertilginoglu wants to merge 4 commits into
Conversation
Not up to standards ⛔🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
- Added WorkflowAiService for detecting deployment jobs using AI. - Integrated AI detection into ProjectSettingsComponent with UI updates. - Created backend service and controller for handling detection requests. - Added necessary DTOs and detection logic using OpenAI. - Updated environment configuration for AI integration.
mertilginoglu
force-pushed
the
feat/deploy-job-detection
branch
from
078a61c to
853b6e7
Compare
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 853b6e7a0f
ℹ️ 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".
| { | ||
| headers: { | ||
| ...(token ? { Authorization: `Bearer ${token}` } : {}), | ||
| ...(currentRepositoryId ? { 'X-Repository-Id': String(currentRepositoryId) } : {}), |
There was a problem hiding this comment.
This request builds the URL from the repositoryId argument but sets X-Repository-Id from repositoryService.currentRepositoryId(), so the path and authorization context can diverge. When they differ (for example during route transitions or if this service is reused with an explicit repository id), the backend computes roles from the header and can authorize against the wrong repository or return 403 for a valid maintainer request. Build the header from the same repositoryId argument used in the URL to keep permission checks consistent.
Useful? React with 👍 / 👎.
| const workflowId = this.detectingWorkflowId(); | ||
| if (workflowId == null) { | ||
| return; |
There was a problem hiding this comment.
The mutation callbacks read and clear a single shared detectingWorkflowId, which races when users trigger detection on multiple workflows before earlier calls settle. In that case, one request can overwrite the id of another, causing errors to be recorded under the wrong workflow (or skipped entirely when onSettled from another call sets it to null first). Use per-invocation variables/context (onError(_, vars) and onSettled(_, _, vars)) or a per-workflow pending map instead of shared global state.
Useful? React with 👍 / 👎.
| {}, | ||
| { | ||
| headers: { | ||
| ...(token ? { Authorization: `Bearer ${token}` } : {}), |
There was a problem hiding this comment.
We can use openapi here instead of using http.post
helios-staging-aet
Bot
had a problem deploying
to
artemis-test7.artemis.cit.tum.de
|
There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions. |
meryemefe
force-pushed
the
staging-ai-integration
branch
from
abcd835 to
6840695
Compare
meryemefe
force-pushed
the
staging-ai-integration
branch
from
fb96bf0 to
350db1a
Compare
|
There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions. |
2 participants
Motivation
Description
Testing Instructions
Prerequisites:
Flow:
Screenshots
Checklist
General
Server
Client