Skip to content

fix(ci): activate .deepsource.toml JS analyzer config (rename to valid 'javascript' name) - #627

Open
d-oit wants to merge 2 commits into
mainfrom
fix/deepsource-js-analyzer-name
Open

fix(ci): activate .deepsource.toml JS analyzer config (rename to valid 'javascript' name)#627
d-oit wants to merge 2 commits into
mainfrom
fix/deepsource-js-analyzer-name

Conversation

@d-oit

@d-oit d-oit commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Summary

The .deepsource.toml JS analyzer section has been silently ignored because it uses the legacy analyzer name javascript-typescript. Per DeepSource docs, the valid name is javascript (the legacy alias is no longer recognized), and the config must live on the default branch to activate.

Impact (confirmed from run payloads)

  • Suppressions never applied — 7 JS-R1005 complexity issues raised with 0 suppressed, despite issue_patterns entries
  • skip_doc_coverage never applied — Documentation Coverage metric counts all function/class artifacts
  • Effective config falls back to dashboard defaults (module_system: commonjs, threshold: low, style_guide: airbnb) instead of the repo's intended es-modules/critical

Change

  • .deepsource.toml: name = "javascript-typescript"name = "javascript"

No analyzer behavior changes beyond activating the repo's already-committed suppression and doc-coverage configuration.

@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
do-knowledge-studio Ready Ready Preview, v0 Aug 9, 2026 6:24pm

@github-actions github-actions Bot added the config label Aug 9, 2026
@deepsource-io

deepsource-io Bot commented Aug 9, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 63558b9...cd07f77 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Aug 9, 2026 8:19p.m. Review ↗
Python Aug 9, 2026 8:19p.m. Review ↗
Shell Aug 9, 2026 8:19p.m. Review ↗
SQL Aug 9, 2026 8:19p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@codacy-production

Copy link
Copy Markdown
Contributor

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

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.

@owl-watch owl-watch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦉 OwlWatch scanned this change — no actionable findings.

@owl-watch owl-watch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦉 OwlWatch scanned this change — no actionable findings.

@d-oit d-oit closed this Aug 9, 2026
auto-merge was automatically disabled August 9, 2026 18:33

Pull request was closed

@d-oit d-oit reopened this Aug 9, 2026
@d-oit
d-oit enabled auto-merge (squash) August 9, 2026 18:33
@d-oit

d-oit commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

Status check: re-verifying merge-state per Plan 098 — all ruleset gates confirmed green on head commit; auto-merge re-armed.

@d-oit

d-oit commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

Plan 098 last-resort nudge: close/reopen to force GitHub ruleset re-evaluation. All gates are green (Codacy required check, commitlint, DeepSource, 0 threads).

@d-oit d-oit closed this Aug 9, 2026
auto-merge was automatically disabled August 9, 2026 20:19

Pull request was closed

@d-oit d-oit reopened this Aug 9, 2026
@d-oit
d-oit enabled auto-merge (squash) August 9, 2026 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant