Commit Graph

11 Commits

Author SHA1 Message Date
fireddd dd5a4c9600 fix(ci): add continue-on-error to coverage comment step
Fork PR tokens cannot create or update comments on the upstream repo.
Adding continue-on-error prevents the entire coverage job from failing
when comment posting is blocked by GitHub permissions.

Closes #913

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-10 02:28:59 +05:30
fireddd 4844e2eafa fix(ci): handle 403 when updating coverage comment on fork PRs
Fork PR tokens cannot PATCH comments owned by a different integration.
Fall back to creating a new comment when the update fails instead of
failing the entire step.

Closes #913

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-10 02:25:01 +05:30
fireddd 10f906c4d8 Merge remote-tracking branch 'upstream/main' into ci/coverage-report-913
# Conflicts:
#	.github/workflows/coverage.yml
2026-04-10 02:16:53 +05:30
fireddd 23f4bd6bfe fix(ci): prevent Node.js code injection in plugin directory name
Replace `node -e` with `jq` for reading plugin package names in the
coverage workflow. The previous approach interpolated $plugin_dir
directly into a JavaScript expression, which could allow code injection
via crafted directory names.

Closes #913

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-10 02:08:01 +05:30
Prateek 967e864f5a chore: rename @composio scope to @aoagents across all packages
Renames all npm package scopes from @composio/* to @aoagents/* and
updates GitHub repo references from ComposioHQ/agent-orchestrator
to aoagents/ao throughout the codebase.

- All package.json names and dependencies
- README badges, links, and install instructions
- Documentation references
- Changeset config
- Source code imports and test files
2026-04-09 15:59:33 +00:00
fireddd a89951b023 fix: remove dead has_coverage output variable (#913)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-05 17:37:29 +05:30
fireddd 7516929e57 fix: address review feedback — sanitize base_ref and always update comment
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-05 12:26:50 +05:30
fireddd dfae0ee6e5 fix: use --coverage.enabled instead of --coverage flag (#913)
Avoid mixing boolean --coverage flag with dot-notation
--coverage.reporter=json which can conflict in vitest v4.
Use --coverage.enabled for explicit, compatible behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-04 23:12:32 +05:30
fireddd 06fe717d28 ci: add non-blocking test coverage report to PRs (#913)
Replace the blocking diff-cover workflow with a non-blocking coverage
reporter that posts a PR comment showing lines covered, lines not
covered, overall coverage percentage, and uncovered line ranges —
scoped to only the files changed in the PR.

Closes #913

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-04 23:00:23 +05:30
fireddd 68eafe9f0d ci: only run coverage for packages with changed files
Uses dorny/paths-filter to detect which packages have changes.
Tests and coverage only run for affected packages — e.g. a
web-only change won't trigger core or CLI tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-31 09:28:31 +05:30
fireddd e6c3a06427 ci: add diff coverage workflow enforcing 80% on changed code only
Uses diff-cover to check that newly added or modified lines in PRs
have at least 80% test coverage, without requiring the entire
codebase to meet the threshold.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-31 09:26:36 +05:30