mo-needs-to-review?

Determine if uncommitted changes require senior review from Mo.

680|24|Updated Sep 6, 2024
One-click install
npx skills add https://github.com/inline-chat/inline --skill mo-needs-to-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: mo-needs-to-review?
Source: https://github.com/inline-chat/inline/tree/main/.codex/skills/mo-needs-to-review%3F
Command: npx skills add https://github.com/inline-chat/inline --skill mo-needs-to-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps determine whether uncommitted changes in the current working tree require a senior review from Mo before merging or releasing.

Core Features & Use Cases

  • Automated risk assessment: checks for criteria including encryption and security best practices, backward compatibility, production-risk changes, migrations status, environment variables, and test results.
  • Review gating: returns a clear decision to escalate to Mo when risk is detected or to proceed when clear.
  • Use Case: When a change touches core components, migrations, or introduces new environment variables, run this Skill to decide if Mo's review is required before PR.

Quick Start

Run the mo-needs-to-review? skill on the current git workspace to determine if changes require Mo's review before merge.

Frequently Asked Questions about mo-needs-to-review?

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I assess uncommitted code changes for production risk before merging?

To assess uncommitted code changes for production risk, run static checks on your git workspace to evaluate encryption practices, backward compatibility, migrations, and environment variables, returning a clear decision on whether a senior review is required.

What is automated risk assessment for code review and when do I need it?

Automated risk assessment for code review statically analyzes your working tree to detect production-risk changes, unsafe logging, or security gaps. You need it before opening a PR when changes touch core components, database migrations, or introduce new environment variables.

How do I check if database migrations or new environment variables require a senior engineer review?

To check if database migrations or new environment variables require a senior engineer review, run static checks on the current work tree to evaluate migration status and configuration changes, escalating to Mo when risk is detected.

Does this risk assessment work with backend services and feature evolution changes?

Yes, this risk assessment works with backend services and feature evolution changes. It applies static checks aligned with backward compatibility, test results, and security practices to determine if changes require a second manual confirmation.

What specific security and backward compatibility checks are applied to uncommitted changes?

The static checks applied to uncommitted changes include verifying encryption and security best practices, assessing backward compatibility, checking migration status, validating new environment variables, and ensuring the avoidance of unsafe logging.

When should I escalate code changes to Mo for a second manual confirmation?

You should escalate code changes to Mo for a second manual confirmation when the static checks detect risk in production-impacting areas like security practices, backward compatibility, database migrations, or test results within your uncommitted working tree.