Skip to content

Add the jira integration#17

Merged
andrtolv merged 8 commits into
mainfrom
jira-integration
May 14, 2026
Merged

Add the jira integration#17
andrtolv merged 8 commits into
mainfrom
jira-integration

Conversation

@is-yu

@is-yu is-yu commented Apr 24, 2026

Copy link
Copy Markdown

No description provided.

@is-yu is-yu requested a review from andrtolv April 24, 2026 13:50
@is-yu is-yu self-assigned this Apr 24, 2026
@github-actions

github-actions Bot commented Apr 24, 2026

Copy link
Copy Markdown

Sigrid maintainability feedback

✅ You wrote maintainable code and achieved your objective of 3.5 stars.

Show details

Sigrid compared your code against the baseline of 2026-04-24.

👍 What went well?

You fixed or improved 3 refactoring candidates.

Risk System property Location
🔴 Duplication
(Fixed)
sigrid-ext-app/src/app/security/security.html line 21-32
sigrid-ext-app/src/app/maintainability/maintainability.html line 23-34
🔴 Duplication
(Fixed)
sigrid-ext-app/src/app/open-source-health/open-source-health.html line 1-7
sigrid-ext-app/src/app/security/security.html line 1-7
sigrid-ext-app/src/app/maintainability/maintainability.html line 1-7
🔴 Duplication
(Fixed)
sigrid-ext-app/src/app/security/security.html line 1-17
sigrid-ext-app/src/app/maintainability/maintainability.html line 1-17

👎 What could be better?

Unfortunately, 14 refactoring candidates were introduced or got worse.

Risk System property Location
🔴 Duplication
(Introduced)
sigrid-ext-app/src/app/maintainability/maintainability.html line 1-30
sigrid-ext-app/src/app/security/security.html line 1-30
🔴 Duplication
(Introduced)
sigrid-ext-app/src/app/open-source-health/open-source-health.html line 1-18
sigrid-ext-app/src/app/maintainability/maintainability.html line 1-18
sigrid-ext-app/src/app/security/security.html line 1-18
🔴 Duplication
(Introduced)
sigrid-ext-app/src/app/maintainability/maintainability.html line 37-48
sigrid-ext-app/src/app/security/security.html line 35-46
🔴 Duplication
(Introduced)
sigrid-ext-app/src/app/shared/jira-issue-dialog/jira-issue-dialog.scss line 1-29
sigrid-ext-app/src/app/shared/finding-edit/finding-edit.scss line 1-29
🔴 Unit Size
(Introduced)
src/commands/create-jira-issue-command.ts
CreateJiraIssueCommand.buildAdfDescription(JiraFinding[],string)
🟠 Unit Size
(Introduced)
src/commands/create-jira-issue-command.ts
CreateJiraIssueCommand.execute(VsCodeCommandData<CreateJiraIssuePayload>)
🟠 Unit Complexity
(Introduced)
src/commands/create-jira-issue-command.ts
CreateJiraIssueCommand.execute(VsCodeCommandData<CreateJiraIssuePayload>)
🟠 Unit Interfacing
(Introduced)
[src/commands/create-jira-issue-command.ts
CreateJiraIssueCommand.callJiraApi(string,string,string,string,object
⚫️ + 6 more

📚 Remaining technical debt

7 refactoring candidates didn't get better or worse, but are still present in the code you touched.

View this system in Sigrid to explore your technical debt

⭐️ Sigrid ratings

System property System on 2026-04-24 Before changes New/changed code
Volume 5.5 N/A N/A
Duplication 5.2 4.8 4.5
Unit Size 4.5 5.0 4.1
Unit Complexity 3.8 4.7 4.4
Unit Interfacing 4.2 5.3 4.1
Module Coupling 5.2 5.5 5.5
Component Independence 3.6 N/A N/A
Component Entanglement 4.0 N/A N/A
Maintainability 4.5 5.0 4.6

💬 Did you find this feedback helpful?

We would like to know your thoughts to make Sigrid better.
Your username will remain confidential throughout the process.


View this system in Sigrid

Sigrid Open Source Health feedback

✅ You achieved your objective of having no open source vulnerabilities.

✅ You achieved your objective of having no open source libraries with license issues.

Show details

Sigrid compared your code against the baseline of 2026-04-24.


View this system in Sigrid

Sigrid Security feedback (Beta)

✅ You achieved your objective of having no high-severity security findings

Show details

Sigrid CI for Security is currently in Beta. The documentation contains more information on its current state and known limitations.

Sigrid compared your code against the baseline of 2026-04-24 06:56 UTC.

👍 What went well?

You did not introduce any security findings during your changes, great job!

😑 You have remaining security findings

You have 1 open security findings.
You can view these findings in Sigrid.


View this system in Sigrid

Yu Huang and others added 7 commits April 28, 2026 11:14
…port

- Implement `AppResource` service for managing webview URL handling and SVG file loading.
- Introduce reusable `SvgIcon` component for rendering SVG icons.
- Add `webviewBaseUri` command to manage the base URI for webview content.
- Refactor Jira button to use `SvgIcon` with dynamic icon loading.
- Update styling for consistency with VS Code theming.
Add comprehensive unit tests for new services, components, and utilities.
@andrtolv andrtolv merged commit 3c9a595 into main May 14, 2026
2 checks passed
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.

2 participants