Skip to content

Add GitAgent Protocol support (agent.yaml + SOUL.md)#758

Open
computer-agent wants to merge 1 commit into
santifer:mainfrom
computer-agent:gitagent-protocol
Open

Add GitAgent Protocol support (agent.yaml + SOUL.md)#758
computer-agent wants to merge 1 commit into
santifer:mainfrom
computer-agent:gitagent-protocol

Conversation

@computer-agent
Copy link
Copy Markdown

@computer-agent computer-agent commented May 30, 2026

Hi! This PR proposes GitAgent Protocol (GAP) support for career-ops — a small, open standard for portable AI agents (https://gitagent.sh).

Career-ops is already an excellent Claude Code agent. These two files are a lightweight, non-invasive addition that lets it also run on any GAP-compatible runtime and be discovered in the open registry.

What this adds (nothing else changes):

  • agent.yaml — standard manifest capturing the name, version, model (anthropic:claude-sonnet-4-6), 15 named skill modes, runtime limits, and the human_in_the_loop: always compliance flag (faithfully reflecting career-ops' own rule that it never submits without user approval)
  • SOUL.md — career-ops' persona and instructions distilled into the standard soul file, adapted from CLAUDE.md

With these two files, career-ops can run on any GAP-compatible runner and be listed in the open registry at https://registry.gitagent.sh. Totally optional to accept — feel free to tweak or close. Thanks for building career-ops in the open! 🦀


⭐ If GAP looks useful to you, the project lives at https://github.com/open-gitagent/opengap — a star helps more maintainers find the standard.

Summary by CodeRabbit

Release Notes

  • Documentation

    • Added Career-Ops agent documentation outlining behavior guidelines, capabilities (offer evaluation, pipeline tracking, scanning, applications, follow-up), and operational principles.
  • Chores

    • Configured Career-Ops agent with model specifications, runtime parameters, and compliance controls including human-in-the-loop approvals.

Review Change Stack

@github-actions
Copy link
Copy Markdown
Contributor

Welcome to career-ops, @computer-agent! Thanks for your first PR.

A few things to know:

  • Tests will run automatically — check the status below
  • Make sure you've linked a related issue (required for features)
  • Read CONTRIBUTING.md if you haven't

We'll review your PR soon. Join our Discord if you have questions.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

📝 Walkthrough

Walkthrough

This PR introduces the career-ops agent through two complementary files: SOUL.md documents the agent's behavioral identity, principles, capabilities, and operational constraints; agent.yaml configures the agent's metadata, model, skills, runtime parameters, and compliance controls.

Changes

Career-Ops Agent Definition

Layer / File(s) Summary
Agent Soul and Behavioral Guidelines
SOUL.md
Defines the career-ops agent's identity, behavioral principles (human-in-the-loop, honesty, adaptability, learning, data protection), 14 supported operational modes, language support, constraints (no auto-submission, no file overwriting, apply selectively, silent updates), and voice (warm, direct, data-driven, transparent scoring).
Agent Configuration and Runtime Setup
agent.yaml
Configures agent metadata, selects Anthropic Claude Sonnet with generation constraints, enumerates 14 skills modes, sets runtime limits, and defines compliance controls including mandatory human-in-the-loop approval, kill-switch enablement, and PII redaction.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding GitAgent Protocol support through two new files (agent.yaml and SOUL.md), which is confirmed by the PR objectives and file summaries.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@agent.yaml`:
- Around line 10-14: The descriptive skill list text is inconsistent with the
actual skills declared in the skills block: the description claims "14 skill
modes" and lists names like evaluate/compare/contact while the skills block
contains 15 keys (including oferta/ofertas/contacto and interview-prep). Update
the description paragraph to exactly match the skills block: correct the count
to 15 (or remove the extra skill from the skills block if that was unintended),
and replace the enumerated names with the canonical skill keys used in the
manifest (e.g., ensure evaluate, compare, pipeline, PDF, scan, batch, deep,
training, project, tracker, apply, patterns, followup and interview-prep or the
oferta/ofertas/contacto variants match the skills block) so the names and count
are consistent with the skills list.
- Around line 24-39: The skills list in agent.yaml is missing the documented
"latex" mode referenced in SOUL.md; update the skills sequence under the
"skills:" key to include "latex" (lowercase) so the LaTeX export mode is
registered and discoverable by GAP runners and the registry, or alternatively
remove the "latex" entry from SOUL.md if the mode was intentionally deprecated;
locate the "skills" array in agent.yaml and add the "latex" token consistent
with other skill strings.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e2ff9912-ceeb-4bb1-b889-2f1a465b2136

📥 Commits

Reviewing files that changed from the base of the PR and between 831ef7f and f3e3c13.

📒 Files selected for processing (2)
  • SOUL.md
  • agent.yaml

Comment thread agent.yaml
Comment on lines +10 to +14
preparation. Supports 14 skill modes (evaluate, compare, pipeline, PDF, scan,
batch, contact, deep, training, project, tracker, apply, patterns, followup)
and multi-language operation (EN/DE/FR/JA). Designed for quality over
quantity: the agent will not submit applications without explicit user
approval.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Description mode list is inconsistent with the skills block.

The description states "Supports 14 skill modes" and enumerates 14 entries, but the skills list below declares 15. The names also diverge (evaluate/compare/contact here vs. oferta/ofertas/contacto in skills), and interview-prep is missing from this enumeration. Since this manifest is intended for the open registry, align the count and naming with the actual skills keys.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agent.yaml` around lines 10 - 14, The descriptive skill list text is
inconsistent with the actual skills declared in the skills block: the
description claims "14 skill modes" and lists names like
evaluate/compare/contact while the skills block contains 15 keys (including
oferta/ofertas/contacto and interview-prep). Update the description paragraph to
exactly match the skills block: correct the count to 15 (or remove the extra
skill from the skills block if that was unintended), and replace the enumerated
names with the canonical skill keys used in the manifest (e.g., ensure evaluate,
compare, pipeline, PDF, scan, batch, deep, training, project, tracker, apply,
patterns, followup and interview-prep or the oferta/ofertas/contacto variants
match the skills block) so the names and count are consistent with the skills
list.

Comment thread agent.yaml
Comment on lines +24 to +39
skills:
- oferta
- ofertas
- pipeline
- pdf
- scan
- batch
- contacto
- deep
- interview-prep
- training
- project
- tracker
- apply
- patterns
- followup
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Missing latex skill — documented capability not registered.

SOUL.md (capabilities table, Line 38) documents a latex mode ("Export your CV as a polished LaTeX / Overleaf .tex file"), but it is not present in this skills list. As a result, the LaTeX export mode would not be advertised/invokable on GAP runners or in the registry, diverging from the documented behavior. Add it (or remove it from SOUL.md if intentionally dropped).

🔧 Proposed fix to register the documented mode
 skills:
   - oferta
   - ofertas
   - pipeline
   - pdf
+  - latex
   - scan
   - batch
   - contacto
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agent.yaml` around lines 24 - 39, The skills list in agent.yaml is missing
the documented "latex" mode referenced in SOUL.md; update the skills sequence
under the "skills:" key to include "latex" (lowercase) so the LaTeX export mode
is registered and discoverable by GAP runners and the registry, or alternatively
remove the "latex" entry from SOUL.md if the mode was intentionally deprecated;
locate the "skills" array in agent.yaml and add the "latex" token consistent
with other skill strings.

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.

1 participant