AgentSkillsCN

mark-mr-ready

将草稿 MR 标记为可提交评审。移除草稿状态，运行 linting/文档检查，向 Slack 发布消息，更新 Jira 为“正在评审”。适用于用户输入“标记 MR 可提交”、“移除草稿”、“撤销草稿”时使用。

SKILL.md
--- frontmatter
name: mark-mr-ready
description: Mark a draft MR as ready for review. Removes draft status, runs linting/docs check, posts to Slack, updates Jira to In Review. Use when user says "mark MR ready", "remove draft", "undraft".

Mark MR Ready for Review

Inputs

InputTypeDefaultPurpose
mr_idstringrequiredMR ID (e.g., "1459" or "!1459")
projectstringautomation-analytics/automation-analytics-backendGitLab project
issue_keystring-Jira key to update (auto-extracted from MR if empty)
update_jirabooltrueUpdate Jira to In Review
run_lintingbooltrueRun black/flake8 before marking ready
repostring""Repo path for linting (auto-detected)
check_docsbooltrueRun update_docs check

Workflow

1. Bootstrap

  • •persona_load("developer")
  • •knowledge_query(project="automation-analytics-backend", section="gotchas") — MR readiness patterns
  • •check_known_issues("gitlab_mr_update")
  • •Clean mr_id: strip "!" and whitespace

2. Get MR

  • •gitlab_mr_view(project, mr_id) — get details
  • •Parse: title, jira_key (from title or inputs), web_url via extract_mr_url

3. Linting (if run_linting)

  • •Resolve repo from config (match project to gitlab) or inputs.repo
  • •git_format(repo, tool="black", check_only=true) or run black --check
  • •git_lint(repo, tool="flake8", max_line_length=100, ignore="E501,W503,E203")
  • •Block if lint fails — show errors, suggest black . && flake8

4. Docs (if check_docs)

  • •skill_run("update_docs", '{"repo": "...", "check_only": true}') — warn only
  • •persona_load("developer") — restore after update_docs

5. Mark Ready

  • •gitlab_mr_update(project, mr_id, draft=false)
  • •Success if "Updated" or "success" in result

6. Notify & Jira

  • •skill_run("notify_mr", '{"mr_id": X, "project": "...", "issue_key": "..."}') — posts to team Slack
  • •persona_load("developer") — restore after notify_mr
  • •If update_jira and mr_info.jira_key: jira_set_status(issue_key, "In Review")

7. Memory

  • •memory_session_log("Marked MR !X ready for review", web_url)
  • •memory_update("state/current_work", "last_updated", timestamp)
  • •Update open_mrs: set is_draft=false, marked_ready, needs_review=true
  • •Record in learned/patterns.mr_ready_actions

8. Failure Learning

  • •"no such host" → learn_tool_fix("gitlab_mr_update", "no such host", "VPN", "vpn_connect()")
  • •"merge request not found" → check mr_id and project
  • •"not_in_channel" (Slack) → invite bot to team channel

Key MCP Tools

  • •persona_load, gitlab_mr_view, gitlab_mr_update
  • •git_format, git_lint (or lint_python if available)
  • •jira_set_status
  • •knowledge_query, check_known_issues, learn_tool_fix
  • •memory_session_log, memory_update, skill_run