Skip to content

Commit c183aa1

Browse files
committed
Futher clarification regarding short-circuiting cyclic graphs.
1 parent b4df2d1 commit c183aa1

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

docs/community/seps/2084-primitive-groups.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ This SEP recommends flexible grouping membership given diversity in potential us
6565
1. Primitives **can** belong to multiple groups. This is crucial for key use-cases e.g., if grouping tools by specific workflows, a `spell_check` tool might appear in both `compose_email` and `compose_document` groups.
6666
2. Groups, themselves being primitives, **may** also belong to multiple groups. This results in overlapping sets, **not a hierarchy**.
6767
3. Clients **may** interpret transitive relationships based on their specific use-cases (see [reference implementation](#transitivity-example)).
68-
4. Although servers are responsible for avoiding invalid groupings such as self or cyclic memberships, SDKs can help. We argue the maintenance overhead would be modest and consider it acceptable for the additional flexibility, since such features are already implemented in most language compilers / interpreters.
68+
4. Although servers are responsible for avoiding invalid groupings such as self or cyclic memberships, SDKs can help. We argue the maintenance overhead would be trivial and consider it acceptable for the additional flexibility. The reference implementation demonstrates how to easily short-circuit cyclic graphs when using a depth-first search to compose the graph.
6969

7070
## Specification
7171

seps/2084-primitive-groups.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ This SEP recommends flexible grouping membership given diversity in potential us
4848
1. Primitives **can** belong to multiple groups. This is crucial for key use-cases e.g., if grouping tools by specific workflows, a `spell_check` tool might appear in both `compose_email` and `compose_document` groups.
4949
2. Groups, themselves being primitives, **may** also belong to multiple groups. This results in overlapping sets, **not a hierarchy**.
5050
3. Clients **may** interpret transitive relationships based on their specific use-cases (see [reference implementation](#transitivity-example)).
51-
4. Although servers are responsible for avoiding invalid groupings such as self or cyclic memberships, SDKs can help. We argue the maintenance overhead would be modest and consider it acceptable for the additional flexibility, since such features are already implemented in most language compilers / interpreters.
51+
4. Although servers are responsible for avoiding invalid groupings such as self or cyclic memberships, SDKs can help. We argue the maintenance overhead would be trivial and consider it acceptable for the additional flexibility. The reference implementation demonstrates how to easily short-circuit cyclic graphs when using a depth-first search to compose the graph.
5252

5353
## Specification
5454

0 commit comments

Comments
 (0)