Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

@mattjohnsonpint
Copy link
Contributor

Functions that start and stop agents should be exposed as mutations rather than queries on the GraphQL schema, because they modify the state of the system.

We'll use "start" and "stop" as prefixes to identify these.

@mattjohnsonpint mattjohnsonpint requested review from a team and Copilot June 11, 2025 21:14
Copy link
Contributor

Copilot AI left a comment

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 updates the GraphQL mutation conventions to include "start" and "stop" as valid mutation prefixes.

  • Added "start" and "stop" to the list of mutation prefixes in the conventions file.
  • Updated the CHANGELOG to document this new feature.

Reviewed Changes

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

File Description
runtime/graphql/schemagen/conventions.go Added "start" and "stop" to the mutation prefixes list
CHANGELOG.md Documented the new mutation prefixes in the changelog

@mattjohnsonpint mattjohnsonpint enabled auto-merge (squash) June 11, 2025 21:14
@mattjohnsonpint mattjohnsonpint merged commit 004bb58 into main Jun 11, 2025
33 checks passed
@mattjohnsonpint mattjohnsonpint deleted the mjp/mutations branch June 11, 2025 21:16
danstarns added a commit to hypermodeinc/docs that referenced this pull request Jun 12, 2025
danstarns added a commit to hypermodeinc/docs that referenced this pull request Jun 24, 2025
* start change to 0.18

* docs: *

* docs: *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* *

* add events to agents

* *

* *

* *

* add note about the rest of agent fields

* use any over map interface

* use accumulatedReports over context

* more error handling

* pass non null data around

* remove repeated workflow list

* *

* *

* *

* sync to support sub hypermodeinc/modus#875

* *

* add start stop based on hypermodeinc/modus#889

* trunk...

* updates first modus agent

* *

* *

---------

Co-authored-by: Ryan Fox-Tyler <[email protected]>
Co-authored-by: William Lyon <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants