Skip to content

Commit 276f85c

Browse files
committed
fix(arch): permit Server layer to access Value (pre-existing #266 regression)
LlamaArchitectureTest.layeredArchitecture was already failing on main (not introduced by the TTS work): the streaming-completions merge (#266) added LlamaModelBackend (server layer) reads of StopReason / LlamaOutput (value layer), but the Value layer's mayOnlyBeAccessedByLayers list — documented as "the EXACT set of packages that reference it today" — was not updated. Add "Server" to it, the same maintenance the rule's own javadoc prescribes. Unrelated to TTS but folded in here because it blocks PR #268's CI; kept as its own commit so it can be cherry-picked to main independently.
1 parent 7e04296 commit 276f85c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/test/java/net/ladenthin/llama/LlamaArchitectureTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ public class LlamaArchitectureTest {
127127
.whereLayer("Parameters")
128128
.mayOnlyBeAccessedByLayers("Api", "Loader", "Server")
129129
.whereLayer("Value")
130-
.mayOnlyBeAccessedByLayers("Api", "Json", "Parameters")
130+
.mayOnlyBeAccessedByLayers("Api", "Json", "Parameters", "Server")
131131
.whereLayer("Callback")
132132
.mayOnlyBeAccessedByLayers("Api")
133133
.whereLayer("Exception")

0 commit comments

Comments
 (0)