Skip to content

test: add tool registry coverage#983

Merged
OneStepAt4time merged 1 commit intomainfrom
fix/360-core-test-coverage
Apr 3, 2026
Merged

test: add tool registry coverage#983
OneStepAt4time merged 1 commit intomainfrom
fix/360-core-test-coverage

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

@OneStepAt4time OneStepAt4time commented Apr 3, 2026

Summary

Quality Gate

  • tsc --noEmit: PASS
  • npm run build: PASS
  • npm test: PASS

Developed with: v2.10.0
Tested with: v2.10.0

Closes #360

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

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

Version field says "UNAVAILABLE". Per the version gate, this field is required. The server now runs v2.10.0. Please update the PR body with "Developed with: v2.10.0". CI is green otherwise.

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

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

Version field says "UNAVAILABLE". Per the version gate, this field is required. The server now runs v2.10.0. Please update the PR body with "Developed with: v2.10.0" and rebase onto current main. CI is green otherwise.

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

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

Version field says "UNAVAILABLE". Per the version gate, this field is required. The server now runs v2.10.0. Please update the PR body with "Developed with: v2.10.0". CI is green otherwise.

@OneStepAt4time OneStepAt4time force-pushed the fix/360-core-test-coverage branch from f868de0 to 2e37713 Compare April 3, 2026 16:47
@OneStepAt4time
Copy link
Copy Markdown
Owner Author

Addressed requested changes: updated PR version field to Developed with: v2.10.0 and rebased branch on latest main. CI has been retriggered.

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

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

Tool registry test coverage. Approved.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Self-rebase failed — conflicts with #986 (just merged) across hook-settings.ts and hook-settings.test.ts. Please rebase onto current main and push.

@OneStepAt4time OneStepAt4time force-pushed the fix/360-core-test-coverage branch from 2e37713 to 7e1b9ed Compare April 3, 2026 16:57
@OneStepAt4time OneStepAt4time merged commit cbd831e into main Apr 3, 2026
5 checks passed
@OneStepAt4time OneStepAt4time deleted the fix/360-core-test-coverage branch April 3, 2026 16:59
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.

Testing: Missing test coverage for core modules

1 participant