Skip to content

docs(deepsource): inline review threads now resolvable — reply+resolve loop#237

Merged
qnbs merged 2 commits into
mainfrom
docs/deepsource-inline-threads
Jun 24, 2026
Merged

docs(deepsource): inline review threads now resolvable — reply+resolve loop#237
qnbs merged 2 commits into
mainfrom
docs/deepsource-inline-threads

Conversation

@qnbs

@qnbs qnbs commented Jun 24, 2026

Copy link
Copy Markdown
Owner

DeepSource started posting inline PR review comments in resolvable threads (deepsource-io[bot]) on code PRs #235/#236 — the runbook's earlier "no resolveReviewThread step" claim is now obsolete.

This updates docs/DEEPSOURCE-REVIEW-LOOP.md:

  • §1 table + Consequence: findings now also appear as resolvable inline threads → after the fix lands and the analyzer status is green, reply citing the commit + resolveReviewThread so 0 stay unresolved (mirrors CodeAnt).
  • New §3a: GraphQL to list/reply/resolve deepsource-io threads.
  • Corrected the status-vs-check-run reality (JS/Rust/Docker are commit statuses → read commits/$SHA/status + target_url; /check-runs only reliably shows the CSS analyzer).
  • §11 log: DeepSource analyzes tests/unit/ too; JS-C1002 flags const/let (not arrow params); JS-0357 = use-before-define; codecov/patch needs branch tests for new services.

Docs-only.

🤖 Generated with Claude Code

…e loop (§1/§3a/§11)

DeepSource began posting inline PR review comments in resolvable threads
(deepsource-io[bot]) on code PRs #235/#236 — the earlier 'no resolveReviewThread
step' note is obsolete. Update the loop to: fix → status green → reply citing the
commit + resolveReviewThread, 0 unresolved (mirrors CodeAnt). Add §3a GraphQL
snippet, correct the status-vs-check-run reality (read commits/$SHA/status +
target_url), and log the JS-C1002/JS-0357 + tests/unit-is-analyzed + codecov/patch
lessons.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
worldscript-studio Ready Ready Preview, Comment Jun 24, 2026 10:16pm

@deepsource-io

deepsource-io Bot commented Jun 24, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 80ed2e1...9a2963c 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
Docker Jun 24, 2026 10:16p.m. Review ↗
JavaScript Jun 24, 2026 10:16p.m. Review ↗
CSS Jun 24, 2026 10:16p.m. Review ↗
Rust Jun 24, 2026 10:16p.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.

…reads

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@qnbs qnbs merged commit fd41270 into main Jun 24, 2026
22 checks passed
@qnbs qnbs deleted the docs/deepsource-inline-threads branch June 24, 2026 22:42
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.

1 participant