Skip to content

chore(chaining): add AI context for chaining engine (#4824)#6473

Merged
savacano28 merged 8 commits into
mainfrom
issue/add-context-ai-chaining
Jul 1, 2026
Merged

chore(chaining): add AI context for chaining engine (#4824)#6473
savacano28 merged 8 commits into
mainfrom
issue/add-context-ai-chaining

Conversation

@savacano28

@savacano28 savacano28 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

  • Add context for chaining feature

Testing Instructions

  1. Modify chaining code base and check if the context is apply in development or review.

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@savacano28 savacano28 added the filigran team Item from the Filigran team. label Jun 29, 2026
@Filigran-Automation

Copy link
Copy Markdown
Contributor

🤖 [AI-generated]

Hey @savacano28! 👋 Thanks a lot for opening PR #6473 — really appreciate you adding AI context for the chaining engine! 🙏

I just had a quick look and I think the description could be enhanced a little to help reviewers get through it faster. I haven't changed anything in your description — just a gentle, optional suggestion:

Area What could help Suggestion
### Proposed changes Currently empty (two blank bullets) A line or two on what you're adding (the chaining-reviewer agent, chaining-engine instructions, and review skill) and why
### Testing Instructions Still the template placeholder A quick note on how a reviewer can verify the new agent/instructions/skill
### Related issues Still Closes #ISSUE-NUMBER Link the related issue, e.g. Closes #1234 — every PR is expected to be linked to an issue, and the reference is also needed on the PR title

💡 If helpful, the contribution conventions and the PR template walk through what to include.

No rush at all — thanks again for contributing to OpenAEV! 🚀

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.22%. Comparing base (7c430ba) to head (b9b6ea7).

❌ Your project check has failed because the head coverage (2.88%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #6473      +/-   ##
============================================
- Coverage     44.23%   44.22%   -0.01%     
- Complexity     7419     7421       +2     
============================================
  Files          2310     2310              
  Lines         63855    63855              
  Branches       8496     8496              
============================================
- Hits          28244    28242       -2     
- Misses        33809    33812       +3     
+ Partials       1802     1801       -1     
Flag Coverage Δ
backend 66.73% <ø> (+0.01%) ⬆️
e2e 18.78% <ø> (-0.28%) ⬇️
frontend 2.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@savacano28 savacano28 changed the title chore(chaining): add ai context for chaining engine chore(chaining): add AI context for chaining engine (#4824) Jun 29, 2026
@savacano28 savacano28 force-pushed the issue/add-context-ai-chaining branch from 4a84b22 to 35d26ce Compare June 29, 2026 13:33
@savacano28 savacano28 marked this pull request as ready for review June 29, 2026 14:00
Copilot AI review requested due to automatic review settings June 29, 2026 14:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds GitHub Copilot guidance for OpenAEV’s Chaining Engine by introducing dedicated instructions, a review skill, and a specialized reviewer agent, and wiring them into the existing Copilot/agent indexes.

Changes:

  • Add a new Chaining Engine instruction file (.github/instructions/chaining-engine.instructions.md) with architecture notes, invariants, and anti-patterns.
  • Add a new review skill (.github/skills/review-chaining-engine/SKILL.md) and a dedicated agent definition (.github/agents/chaining-engine-reviewer.agent.md).
  • Update AGENTS.md and .github/instructions/code-review.instructions.md to reference the new Chaining Engine guidance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
AGENTS.md Adds Chaining Engine to the “Where to find conventions” table, adds the new skill/agent routing entry, and includes a minor formatting change.
.github/skills/review-chaining-engine/SKILL.md New step-by-step review checklist and guidance for modifying/reviewing chaining engine code.
.github/instructions/code-review.instructions.md Adds a Chaining Engine section linking to the new instructions/skill/agent.
.github/instructions/chaining-engine.instructions.md New Copilot instruction set for chaining engine code paths (API/service/model/scheduler/frontend).
.github/agents/chaining-engine-reviewer.agent.md New specialized agent definition for chaining engine reviews.

Comment thread AGENTS.md Outdated
Comment thread .github/agents/chaining-engine-reviewer.agent.md Outdated
Comment thread .github/agents/chaining-engine-reviewer.agent.md
@savacano28 savacano28 force-pushed the issue/add-context-ai-chaining branch from 365dc67 to b9b6ea7 Compare June 30, 2026 16:45
@savacano28 savacano28 merged commit f588b02 into main Jul 1, 2026
36 checks passed
@savacano28 savacano28 deleted the issue/add-context-ai-chaining branch July 1, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants