Architect round 8: services return data, not flags - #30
Merged
Merged
Conversation
Three rules from a sponsor coding session, recorded generic in the skill spec (D1-D3) and folded into the architect agent definition: services return data and never a flag that steers the caller; derived facts are queried, never stored or passed; a view is the rule and a row in it is the fact. Plugin version 0.13.0 -> 0.13.1.
toaditi
self-requested a review
September 2, 2026 14:23
toaditi
approved these changes
Sep 2, 2026
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
2 participants
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.
What
Three rules the sponsor set in a coding session on 2026-09-02, recorded in
docs/architect-skill-spec.mdas round 8 (D1 to D3) and folded intoagents/moqui-architect.mdunder Rules. Plugin version bumped to 0.14.0 in.claude-plugin/plugin.json(main is 0.11.0; open PRs #23 and #25 already claim 0.12.0 and 0.13.0).Conclusion recorded with them: a service is a step between two database states; anything it holds in memory dies with the call.
Wording
Generic. No client, project, or system names. The sponsor's own words are quoted as evidence.
Not included
The engagement record and code evidence stay in the private coach repo. No eval fixture is added in this PR; the architect fixtures AEC1 and AEC2 remain owed, as the coach pack already records.
Note on the first commit
Its message says "0.13.0 -> 0.13.1". That edit matched nothing; the second commit carries the real bump.