Skip to content

[DO NOT MERGE] eval #31601 i:1: Textual definition update: protein carrier activity and unfolded protein holdase activity (opencode/openai/gpt-5.4, .) - #664

Open
cmungall wants to merge 4 commits into
eval-base-issue-31601from
scribe-v1-ai4curation-go-ontology-agent-config-v9-.-openai-gpt-5.4-iter1-issue-31601
Open

[DO NOT MERGE] eval #31601 i:1: Textual definition update: protein carrier activity and unfolded protein holdase activity (opencode/openai/gpt-5.4, .)#664
cmungall wants to merge 4 commits into
eval-base-issue-31601from
scribe-v1-ai4curation-go-ontology-agent-config-v9-.-openai-gpt-5.4-iter1-issue-31601

Conversation

@cmungall

Copy link
Copy Markdown
Member

⚠️ EVALUATION ONLY - DO NOT MERGE ⚠️

Original Issue

#31601: Textual definition update: protein carrier activity and unfolded protein holdase activity

Original PR (human solution)

#32007: Revise GO:0140597 (protein carrier activity) definition per #31601

Agent Commits

  • Revise protein carrier activity definition
  • Merge pull request #32006 from geneontology/dragon-ai-agent-issue-31963-run7919

Note: This PR diff shows ONLY the agent's changes. The baseline is eval_base_branch.

Agent Response - PR Comments

This PR updates the textual definition for GO:0140597 (protein carrier activity) in response to issue #31601.

Change made:

  • Revised the definition from:
    Binding to and carrying a protein to an acceptor molecule or to a specific location by moving along with the target protein.
  • To:
    Directly binding to a protein and delivering it either to an acceptor molecule or to a specific location.

Rationale:

  • This follows the requested wording from the issue discussion, specifically to avoid the older, more restrictive phrasing.
  • The revised text also aligns better with the wording already used in related carrier activity definitions elsewhere in GO (for example, lipid and ion/molecule carrier activity terms that use the same "Directly binding ... and delivering it either to an acceptor molecule or to a specific location" pattern).
  • No logical axioms or hierarchy changes were needed for this issue; this is a textual clarification only.

Checks completed:

  • PLAN: Issue context reviewed and intent was clear
  • PRE-VALIDATION: Ran pre-edit validation checks relevant to definitions/namespaces
  • TERM-SEARCH: Reviewed current stanzas for GO:0140597 and related GO:0140309
  • DESIGN-PATTERNS: Checked for relevant existing wording/pattern precedent in GO
  • EDITS: Used obo-checkout.pl / obo-checkin.pl workflow via terms/
  • RELATIONSHIPS: Confirmed no relationship or logical definition changes were required
  • METADATA: No metadata changes required for an edit to an existing term
  • AUTOMATED-VALIDATION: Ran robot verify SPARQL QC suite successfully after the edit
  • AUTOMATED-VALIDATION: Ran ELK reasoning successfully after the edit
  • REFERENCE-VALIDATION: Existing PMID retained; no new references introduced

Files changed:

  • src/ontology/go-edit.obo

Local commit:

  • 378528a Revise protein carrier activity definition

🤖 Generated by opencode agent

  • Runtime: opencode
  • Model: openai/gpt-5.4
  • Agent config: ai4curation/go-ontology-agent-config@v9:.
  • Iteration: 1
  • Run: View workflow run

Agent Response - Issue Comments

Updated GO:0140597 (protein carrier activity) to the broader wording requested in the thread.

The definition now reads: "Directly binding to a protein and delivering it either to an acceptor molecule or to a specific location."

The change has been committed locally and can be included in PR #.


🤖 Generated by opencode agent

  • Runtime: opencode
  • Model: openai/gpt-5.4
  • Agent config: ai4curation/go-ontology-agent-config@v9:.
  • Iteration: 1
  • Run: View workflow run

Experiment Config

Parameter Value
Model openai/gpt-5.4
Agent config ai4curation/go-ontology-agent-config@v9:.
Iteration 1
URL prefix https://href.li/?
Base branch eval-base-issue-31601

See workflow artifacts for full agent trace.

dragon-ai-agent[bot] and others added 4 commits April 28, 2026 16:52
…tivity)

Per sjm41 comments on issue #31963:
- Reaction simplified to use 'phytyl diphosphate' and 'geranylgeranyl
  diphosphate' (matching EC:1.3.1.83 / RHEA:26229 wording)
- NADP corrected to NADP+
- Added sentence noting that this enzyme also catalyzes the reduction
  of geranylgeranyl-chlorophyll a into phytyl-chlorophyll a (PMID:9492312)
- Definition references updated to EC:1.3.1.83, PMID:9492312, RHEA:26229

Obsoletion of GO:0045550 to be handled in a separate ticket.

Refs geneontology/go-ontology#31963

---
🤖 Generated by @dragon-ai-agent
- Model: claude-opus-4-7
- Agent harness: claude-code
- Triggered by: @raymond91125
- Run: https://github.com/geneontology/go-ontology/actions/runs/25065339565

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…63-run7919

Update GO:0102067 definition per sjm41 comments (refs #31963)
Model: openai/gpt-5.4
Issue repo: geneontology/go-ontology
Agent config: ai4curation/go-ontology-agent-config@v9:.
Iteration: 1

Artifacts: https://github.com/ai4curation/eval-ont-agent-go/actions/runs/25978277382#artifacts
@cmungall

Copy link
Copy Markdown
Member Author

📋 Traces: traces/25978277382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants