Rename Toggle Endpoint to State for Clear and Explicit Status Updates(issue1497)#2092
Merged
crivetimihai merged 1 commit intomainfrom Jan 19, 2026
Merged
Rename Toggle Endpoint to State for Clear and Explicit Status Updates(issue1497)#2092crivetimihai merged 1 commit intomainfrom
crivetimihai merged 1 commit intomainfrom
Conversation
9105a8f to
eeee1f9
Compare
Closes #1497 Changes: - Rename all /toggle endpoints to /state - Add deprecated /toggle aliases that redirect to /state - Update LLM provider/model state methods to accept explicit activate parameter - Add GET /resources/{resource_id}/info endpoint - Update gRPC /state endpoint to accept explicit activate parameter - Fix admin UI gRPC form paths to include /admin prefix - Update admin UI to pass explicit state for gRPC services - Update audit trail actions and structured logging event types - Update error messages and log messages - Update test function names from toggle to state for consistency - Update locust load test function names for consistency - Update docs: ADR-011 and code2flow.svg references Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
eeee1f9 to
11dce70
Compare
crivetimihai
approved these changes
Jan 19, 2026
Member
Changes SummaryThis PR has been rebased, reviewed, and updated with the following changes: Core API Changes
Admin UI
Audit & Logging
Tests
Documentation
Code Quality
|
gabe-l-hart
added a commit
to contextforge-org/contextforge-cli
that referenced
this pull request
Jan 27, 2026
This was refactored in IBM/mcp-context-forge#2092 Branch: RemoveToggle Signed-off-by: Gabe Goodhart <ghart@us.ibm.com>
kcostell06
pushed a commit
to kcostell06/mcp-context-forge
that referenced
this pull request
Feb 24, 2026
IBM#2092) Closes IBM#1497 Changes: - Rename all /toggle endpoints to /state - Add deprecated /toggle aliases that redirect to /state - Update LLM provider/model state methods to accept explicit activate parameter - Add GET /resources/{resource_id}/info endpoint - Update gRPC /state endpoint to accept explicit activate parameter - Fix admin UI gRPC form paths to include /admin prefix - Update admin UI to pass explicit state for gRPC services - Update audit trail actions and structured logging event types - Update error messages and log messages - Update test function names from toggle to state for consistency - Update locust load test function names for consistency - Update docs: ADR-011 and code2flow.svg references Signed-off-by: Mihai Criveti <crivetimihai@gmail.com> Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
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.
closes issue #1497
This pull request standardizes API endpoints and related documentation by replacing the verb "toggle" with "state" for activating or deactivating resources across the codebase, documentation, and admin UI. This change improves clarity and consistency for users and developers interacting with the system.
API and Endpoint Changes
/toggleto/state, clarifying that the endpoint sets the state rather than simply toggling it. [1] [2] [3]Documentation Updates
README.md,docs/docs/index.md,docs/docs/manage/api-usage.md,docs/docs/using/grpc-services.md) have been updated to use the new/stateendpoint in example API calls, ensuring instructions match the codebase. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]Architecture and ADR Updates
Code and Test Refactoring
set_server_stateinstead oftoggle_server_status), improving code clarity and alignment with endpoint changes. [1] [2]These changes collectively enhance the clarity and maintainability of both the API and its documentation, making it easier for users and developers to understand and interact with resource state management.