docs(specs): update SPEC-PLATFORM-SERVICE post v1.1.0#461
Merged
omid-aignostics merged 1 commit intomainfrom Mar 6, 2026
Merged
docs(specs): update SPEC-PLATFORM-SERVICE post v1.1.0#461omid-aignostics merged 1 commit intomainfrom
omid-aignostics merged 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates specifications to reflect recent Platform API/environment changes and extends the Utils specification with a plugin system, adding integration test scaffolding (dummy plugin + tests) to validate plugin discovery/registration.
Changes:
- Update
SPEC_PLATFORM_SERVICEto include thetestenvironment and documentApplicationRun.results()filtering parameters. - Extend
SPEC-UTILS-SERVICE+ add new requirements/features around plugin discovery and CLI/GUI registration. - Add integration tests + a dummy plugin package used to verify plugin auto-discovery (and update pre-commit exclusion for test resources).
Reviewed changes
Copilot reviewed 18 out of 19 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
uv.lock |
Updates locked dependency versions (authlib, lxml-html-clean). |
specifications/SPEC_PLATFORM_SERVICE.md |
Docs/spec updates for environments and ApplicationRun.results() signature. |
specifications/SPEC-UTILS-SERVICE.md |
Adds FRs for plugin discovery + CLI/GUI registration. |
requirements/SHR-UTILS-1.md |
Refines requirement wording/scope for MCP server audience and functionality. |
requirements/SHR-UTILS-2.md |
Introduces high-level requirement for SDK plugin extensibility. |
requirements/SWR-UTILS-2-1.md |
Adds requirement for plugin module discovery/loading. |
requirements/SWR-UTILS-2-2.md |
Adds requirement for plugin CLI command integration. |
requirements/SWR-UTILS-2-3.md |
Adds requirement for plugin GUI page integration. |
tests/resources/mcp_dummy_plugin/pyproject.toml |
Dummy plugin package metadata + entry point registration for tests. |
tests/resources/mcp_dummy_plugin/src/mcp_dummy_plugin/_cli.py |
Dummy Typer CLI contributed by plugin. |
tests/resources/mcp_dummy_plugin/src/mcp_dummy_plugin/_nav.py |
Dummy GUI nav builder contributed by plugin. |
tests/resources/mcp_dummy_plugin/src/mcp_dummy_plugin/__init__.py |
Exposes dummy plugin CLI/nav for discovery. |
tests/aignostics/utils/conftest.py |
Adds session fixture to install/uninstall dummy plugin for integration tests. |
tests/aignostics/utils/mcp_test.py |
Reuses shared plugin-install fixture; removes in-test install logic. |
tests/aignostics/utils/plugin_test.py |
New integration tests for plugin CLI + GUI nav registration/discovery. |
tests/aignostics/utils/di_test.py |
Updates recorded spec ID metadata for DI tests. |
tests/aignostics/utils/TC-UTILS-PLUGIN-02.feature |
Feature spec for plugin CLI registration. |
tests/aignostics/utils/TC-UTILS-PLUGIN-03.feature |
Feature spec for plugin GUI nav registration. |
.pre-commit-config.yaml |
Excludes tests/resources/ from the name-tests-test hook. |
706e932 to
629185b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
testto the list of supported deployment environments (production, staging, development, test), reflecting the test environment constants and settings routing added in chore: add test environment support #422.ApplicationRun.results(): Updated the public API signature to document theitem_idsandexternal_idsfilter parameters introduced in feat(platform): Add item filtering for fetching run results. #442.Test plan