add module that uses LLM api#8
Merged
Merged
Conversation
Signed-off-by: Connor Braa <connor@dagger.io>
this way we can git-call it from dagger/dagger and expect failures, but we also get the nested git-calls-dir thing in llm-dir-module-depender Signed-off-by: Connor Braa <connor@dagger.io>
Signed-off-by: Connor Braa <connor@dagger.io>
Signed-off-by: Connor Braa <connor@dagger.io>
5 tasks
Signed-off-by: Connor Braa <connor@dagger.io>
Signed-off-by: Connor Braa <connor@dagger.io>
cwlbraa
commented
Mar 21, 2025
| } | ||
|
|
||
| func (m LlmTestModule) llm(stringArg string) *dagger.LLM { | ||
| return dag.Llm(dagger.LlmOpts{Model: m.Model}).WithPrompt(stringArg).SetString("CACHE_BUSTER", uuid.NewString()) |
Contributor
Author
There was a problem hiding this comment.
this was kinda a shot in the dark, but i like to re-use the same prompt under test and I think this gives the cache-busting behavior necessary to do that without n=2+ calls always succeeding due to cache hits.
vito
approved these changes
Mar 21, 2025
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.
this will help us test LLM-allow policies.
there are 2 modules here, the outer one depends on the inner one via relative-path import. this is to test that we don't have a hole in our policy if GitModuleSources import DirModuleSources. Spoiler, there's no hole afaict, the inner one gets treated as a GitModuleSource too.