Files
drug-discovery-prompts/upstream/mims-harvard-ToolUniverse/skills/devtu-code-optimization/references/code-patterns.md

216 lines
6.0 KiB
Markdown

---
title: "Code Patterns Reference"
task: ""
lineage_type: import
upstream_source: https://github.com/mims-harvard/ToolUniverse/blob/3038dcbe/skills/devtu-code-optimization/references/code-patterns.md
upstream_sha: 3038dcbe
imported_at: 2026-06-30
prompt_class: prompt
upstream_changes: accepted
author: upstream
validated: false
---
# Code Patterns Reference
Reusable implementation patterns for ToolUniverse tool development.
## Schema Patterns
### return_schema with oneOf (required)
```json
{
"return_schema": {
"oneOf": [
{
"type": "object",
"properties": {
"data": {"type": "object"},
"metadata": {"type": "object"}
}
},
{
"type": "object",
"properties": {
"error": {"type": "string"}
}
}
]
}
}
```
### Nullable fields
```json
{"type": ["array", "null"]}
{"type": ["string", "null"]}
```
## API Call Patterns
### Client-Side Filter (when API ignores params)
```python
results = api_call(base_params_only)
if interaction_types:
results = [r for r in results if r.get("type") in interaction_types]
if sources:
results = [r for r in results if r.get("source") in sources]
```
### Fallback Lookup
```python
precise = api_call(geneSymbol=gene_symbol)
if not precise:
precise = api_call(name=gene_symbol)
```
### Client-Side Pagination (when API ignores size/page)
```python
all_items = api_call()
start = page * size
return all_items[start:start + size]
```
### PostgREST Join
```python
url = f"{base}/recommendation?select=*,drug(name)&genesymbol={_postgrest_eq(gene)}"
```
## Output Patterns
### Normalization Disclosure
```python
_norm_parts = []
if original != normalized:
_norm_parts.append(f"'{original}' → '{normalized}' (reason)")
if _norm_parts:
result["normalization_note"] = "Auto-normalized: " + "; ".join(_norm_parts)
```
### Truncation at Top Level
```python
response = {"status": "success", "data": data[:limit]}
if len(data) > limit:
response["truncated"] = True
response["truncation_note"] = (
f"Returning {limit} of {len(data)}. "
f"Pass max_results={len(data)} for full data."
)
```
### No-Data vs Bad-Query
```python
if count == 0 and query:
result["hint"] = f"No results for '{query}'. Try a broader term or check spelling."
elif count == 0:
result["hint"] = "No data available for this entity."
```
## Hosted Model-API Tools (NVIDIA NIM-style)
### Async poll host = the invocation host
Poll a 202 job-status on the SAME gateway you POSTed to. NVCF biology NIMs invoke
**and** poll on `health.api.nvidia.com`; `integrate.api.nvidia.com` serves only the
OpenAI-compatible LLM endpoints and has **no** `/v1/status` route.
```python
host = urlparse(self.base_url).netloc # e.g. health.api.nvidia.com
poll_url = f"https://{host}/v1/status/{req_id}"
```
### Route-existence probe (find/verify hosted endpoints)
Plain-text `404 page not found` = route does NOT exist; a structured
`{"status":404,...}` (or 400/422/200) = route exists. Use the live API to confirm a
model is hosted and to find the right slug before wrapping it.
### Unwrap JSON envelopes around the "raw" payload
Some endpoints return `{"pdbs": ["...ATOM..."]}` even when response_type is `pdb`.
Unwrap to the inner value so the field matches the schema (real PDB, not a JSON blob).
### HTTP 200 with an inner failure
A 200 can carry `{"status": "failed", ...}` (e.g. DiffDock with an unreadable
ligand). Surface it as an error — but only on explicit `failed/error/errored`; an
inner `status:"success"` must stay a success (don't over-match).
### 404 "not found for account" ≠ wrong path
A gated/unprovisioned model returns a 404 whose body says "Not found for account".
Report "model not available for your account" rather than "endpoint not found".
### Retry a longer poll window before declaring "broken"
A heavy async job can return 504 / `nvcf-status: errored` simply because
`NVCF-POLL-SECONDS` was shorter than its runtime. Only a *persistent* 400
`DEGRADED`/error across retries is a real outage. 5xx bodies are often empty —
surface `nvcf-status` / `nvcf-reqid` from the headers instead.
### Model-variant selection via templated endpoint
Expose multiple hosted sizes through one tool: a `{placeholder}` in the endpoint +
`fields.path_params` default, filled from the request arg (sanitized slug) and
stripped from the request body.
```python
# endpoint "arc/{model}/generate", path_params {"model": "evo2-40b"}
value = args.get(key) or default
if not re.fullmatch(r"[A-Za-z0-9._-]+", str(value)): # no path injection
value = default
endpoint = endpoint.replace("{" + key + "}", value)
```
## R-subprocess Tools
### Run a script file, not `Rscript -e <string>`
`Rscript -e` collapses one backslash level before R parses it, so a regex literal
like `sub("\\..*", ...)` becomes `sub("\..*", ...)` and R aborts with
`'\.' is an unrecognized escape`. Write the script to a temp `.R` file and run
`Rscript <file>` (parsed verbatim); always remove the temp file, incl. on timeout.
```python
tmp = tempfile.NamedTemporaryFile(mode="w", suffix=".R", delete=False)
try:
tmp.write(r_script); tmp.close()
return subprocess.run(["Rscript", tmp.name], capture_output=True, text=True, timeout=t)
finally:
try: os.unlink(tmp.name)
except OSError: pass
```
## try/except Indentation (Critical)
```python
# CORRECT
try:
resp = requests.get(url)
data = resp.json()
except Exception as e:
return {"status": "error", "error": str(e)}
# WRONG — SyntaxError
try:
resp = requests.get(url)
if resp.ok: # ← same indent as try: → OUTSIDE try block
data = resp.json()
except Exception: # ← Python: "try without except"
pass
```
Every `try:` must have `except:` at the **exact same indentation level**.
## Multi-Word Search Hint
```python
if result["count"] == 0 and name_q and " " in str(name_q):
first_word = str(name_q).split()[0]
result["multi_word_hint"] = (
f"Search may not match multi-word phrases like '{name_q}'. "
f"Try a single keyword: name='{first_word}'."
)
```