Skip to content

Add DevDigest CI review - #30

Merged
burnjohn merged 1 commit into
mainfrom
devdigest/ci
Sep 10, 2026
Merged

burnjohn merged 1 commit into
mainfrom
devdigest/ci

Conversation

@burnjohn

Copy link
Copy Markdown
Owner

This PR adds DevDigest's automated review (agents: General Reviewer, Spec Conformance Reviewer) as a GitHub Actions workflow. Add the OPENROUTER_API_KEY repository secret before merging so the workflow can run.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 36 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 317ea9f1-3148-4b20-abab-a5d39dad2e9e

📥 Commits

Reviewing files that changed from the base of the PR and between 2a11d58 and 4352bb1.

📒 Files selected for processing (4)
  • .devdigest/agents/general-reviewer-9833590c.yaml
  • .devdigest/agents/spec-conformance-reviewer-4336cb79.yaml
  • .devdigest/runner/300.index.js
  • .devdigest/runner/index.js

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions 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.

DevDigest — 2 reviewers

✅ General Reviewer · ✅ Spec Conformance Reviewer


General Reviewer — Approved ✅

No findings. Looks good.


Spec Conformance Reviewer — Approved ✅

No findings. Looks good.

@burnjohn
burnjohn merged commit 1d85e77 into main Sep 10, 2026
6 of 7 checks passed
@greptile-apps

greptile-apps Bot commented Sep 10, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The PR should not merge until the spec-conformance reviewer receives the specification it is expected to evaluate.

Findings

  1. P1 Specification input is missing ▶

Summary

  • Renames the existing general-reviewer manifest.
  • Adds a specification-conformance agent using OpenRouter.
  • Extends the runner to load all manifests, run them sequentially, merge reviews, and report per-agent artifact data.
  • Regenerates the runner’s dynamic chunk and module identifiers.

Diagram

sequenceDiagram
  participant CI as GitHub Actions
  participant Runner as DevDigest Runner
  participant General as General Reviewer
  participant Spec as Spec Reviewer
  participant GH as GitHub Review API
  CI->>Runner: Run with PR context and credentials
  Runner->>General: Review(diff, PR description, skills)
  General-->>Runner: Findings and verdict
  Runner->>Spec: Review(diff, PR description, empty skills)
  Note over Spec: No specification is supplied
  Spec-->>Runner: Ungrounded conformance result
  Runner->>Runner: Merge agent results
  Runner->>GH: Post one combined review
Loading

Reviews (1) · Last reviewed commit: "Update DevDigest CI configuration"

secrets

or data integrity. A wrong number on a dashboard card is a WARNING.
skills: []

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Specification input is missing

When a PR description does not contain the complete specification, this agent runs with empty skills and no specs input, so it cannot ground the required acceptance-criterion verdicts and can post fabricated or incomplete conformance findings.

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