generated from coulomb/repo-seed
Compare commits
1 Commits
main
...
codex/wp-0
| Author | SHA1 | Date | |
|---|---|---|---|
| 9709692db8 |
@@ -2,7 +2,7 @@
|
||||
|
||||
Registered as source type 'repo-scoping'.
|
||||
Supported queries:
|
||||
- repo_profile: GET {REPO_SCOPING_URL}/repos/{repo_slug}/scope
|
||||
- repo_profile: GET {REPO_SCOPING_URL}/repos/{repo_slug}/scope/context
|
||||
|
||||
5-minute in-process cache keyed by (query, repo_slug). Cache is per-worker-
|
||||
process; not shared across Temporal workers.
|
||||
@@ -36,7 +36,7 @@ class RepoScopingContextResolver(ContextResolver):
|
||||
ts, val = _CACHE[cache_key]
|
||||
if now - ts < _CACHE_TTL:
|
||||
return val
|
||||
url = f"{_REPO_SCOPING_URL.rstrip('/')}/repos/{repo_slug}/scope"
|
||||
url = f"{_REPO_SCOPING_URL.rstrip('/')}/repos/{repo_slug}/scope/context"
|
||||
resp = httpx.get(url, timeout=10.0)
|
||||
resp.raise_for_status()
|
||||
result: dict[str, Any] = resp.json()
|
||||
|
||||
66
tests/test_repo_scoping_context_resolver.py
Normal file
66
tests/test_repo_scoping_context_resolver.py
Normal file
@@ -0,0 +1,66 @@
|
||||
from activity_core.context_resolvers import repo_scoping
|
||||
|
||||
|
||||
class Response:
|
||||
def __init__(self, body):
|
||||
self.body = body
|
||||
|
||||
def raise_for_status(self):
|
||||
return None
|
||||
|
||||
def json(self):
|
||||
return self.body
|
||||
|
||||
|
||||
def test_repo_scoping_context_resolver_calls_scope_context_endpoint(monkeypatch):
|
||||
calls = []
|
||||
body = {
|
||||
"repo_slug": "repo-scoping",
|
||||
"capabilities": ["Generate SCOPE.md"],
|
||||
"tags": ["api", "scope"],
|
||||
"scope_md_exists": True,
|
||||
"scope_summary": "Maps repositories into reviewable context.",
|
||||
}
|
||||
|
||||
def fake_get(url, timeout):
|
||||
calls.append((url, timeout))
|
||||
return Response(body)
|
||||
|
||||
repo_scoping._CACHE.clear()
|
||||
monkeypatch.setattr(repo_scoping, "_REPO_SCOPING_URL", "http://repo-scoping.local/")
|
||||
monkeypatch.setattr(repo_scoping.httpx, "get", fake_get)
|
||||
|
||||
resolver = repo_scoping.RepoScopingContextResolver()
|
||||
result = resolver.resolve(
|
||||
"repo_profile",
|
||||
None,
|
||||
{"repo_slug": "repo-scoping"},
|
||||
)
|
||||
|
||||
assert result == body
|
||||
assert calls == [
|
||||
(
|
||||
"http://repo-scoping.local/repos/repo-scoping/scope/context",
|
||||
10.0,
|
||||
)
|
||||
]
|
||||
|
||||
cached = resolver.resolve(
|
||||
"repo_profile",
|
||||
None,
|
||||
{"repo_slug": "repo-scoping"},
|
||||
)
|
||||
|
||||
assert cached == body
|
||||
assert len(calls) == 1
|
||||
|
||||
|
||||
def test_repo_scoping_context_resolver_ignores_unknown_queries(monkeypatch):
|
||||
repo_scoping._CACHE.clear()
|
||||
monkeypatch.setattr(
|
||||
repo_scoping.httpx,
|
||||
"get",
|
||||
lambda *args, **kwargs: (_ for _ in ()).throw(AssertionError("unexpected HTTP")),
|
||||
)
|
||||
|
||||
assert repo_scoping.RepoScopingContextResolver().resolve("unknown", None, {}) == {}
|
||||
@@ -601,7 +601,7 @@ to `context[source.bind_to]`. A resolver that raises logs a warning and binds
|
||||
Registered as source type `repo-scoping`.
|
||||
|
||||
Supported queries:
|
||||
- `repo_profile`: `GET {REPO_SCOPING_URL}/repos/{params['repo_slug']}/scope`
|
||||
- `repo_profile`: `GET {REPO_SCOPING_URL}/repos/{params['repo_slug']}/scope/context`
|
||||
Returns dict with `capabilities`, `tags`, `scope_summary`, `scope_md_exists`.
|
||||
|
||||
5-minute in-process cache keyed by `(query, repo_slug)`. Cache is per-worker-
|
||||
|
||||
@@ -123,7 +123,7 @@ to `context[source.bind_to]`. A resolver that raises logs a warning and binds
|
||||
Registered as source type `repo-scoping`.
|
||||
|
||||
Supported queries:
|
||||
- `repo_profile`: `GET {REPO_SCOPING_URL}/repos/{params['repo_slug']}/scope`
|
||||
- `repo_profile`: `GET {REPO_SCOPING_URL}/repos/{params['repo_slug']}/scope/context`
|
||||
Returns dict with `capabilities`, `tags`, `scope_summary`, `scope_md_exists`.
|
||||
|
||||
5-minute in-process cache keyed by `(query, repo_slug)`. Cache is per-worker-
|
||||
|
||||
Reference in New Issue
Block a user