Run any Skill in Manus with one click

coding-standards

Stars23

Forks7

UpdatedMarch 20, 2026 at 06:31

Load perl-lsp coding standards for code-writing or code-review work in this repo. Use when implementing, fixing, reviewing, or validating code.

Installation

Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.

Run Skill in Manus

Source

EffortlessMetrics

EffortlessMetrics/perl-lsp

View GitHub Repository View Creator Repositories

Download

Run Skill in Manus

Related occupationsSOC

Based on SOC occupation classification

Software DevelopersComputer and Mathematical Occupations·SOC 15-1252

SKILL.md

readonly

name	coding-standards
description	Load perl-lsp coding standards for code-writing or code-review work in this repo. Use when implementing, fixing, reviewing, or validating code.
user-invocable	false

Coding Standards

This skill is the full on-demand coding standards reference for agents. Hooks may inject a condensed summary automatically, but that Layer 0 context is not a replacement for the complete standards below.

Banned In Production Code

unwrap(), expect() → use ?, .ok_or_else(), or pattern matching
panic!(), todo!(), unimplemented!() → return Result/Option
dbg!() → use tracing::debug!
std::process::abort() → never
std::process::exit() → only in bin/ directories and lifecycle.rs
Exception: one #[allow(clippy::expect_used)] in crates/perl-lsp/src/util/uri.rs

Patterns

Regex init: Option<Regex> with .ok() for graceful degradation
Non-empty collections: fixed-size arrays [T; N] for compile-time guarantees
.first() over .get(0)
.push(char) not .push_str("x") for single chars
or_default() not or_insert_with(Vec::new)
No unnecessary .clone() on Copy types

Test Standards

Return Result<()> from tests, or use perl_tdd_support::must / must_some
Descriptive names: test_<what>_<scenario>_<expected>
LSP tests: RUST_TEST_THREADS=2 cargo test -p perl-lsp -- --test-threads=2

Commit Format

Conventional: type(scope): description
Types: fix, feat, test, docs, chore, perf, refactor
Scope: crate name, subsystem, or repo area being changed

Verification

Run cargo fmt --all && cargo clippy -p <crate> --tests -- -D warnings && cargo test -p <crate>

Escalate to nix develop -c just ci-gate for changes spanning 3+ crates.

Git Commit Hygiene

Only git add files you intentionally created or modified
Never use git add -A or git add .
Exclude from commits unless your task requires them:
- Cargo.lock unless dependencies changed
- .claude/ infrastructure files unless that is the task
- docs/project/CURRENT_STATUS.md
- scripts/.ignored-baseline
Before committing, run git diff --cached --name-only

Dual Indexing

file_index.references.entry(bare_name.to_string()).or_default().push(symbol_ref.clone());
file_index.references.entry(qualified).or_default().push(symbol_ref);

More from this repository

same repository

merge-queue

EffortlessMetrics/perl-lsp

Merge approved PRs in safe batches — checks CI, fixes policy failures, handles conflicts, ratchets corpus after parser merges.

2026-03-2023

parser-fix

EffortlessMetrics/perl-lsp

Run a TDD parser-fix workflow for perl-parser-core or related parser crates. Use when a parser bug has a concrete failing construct and needs tests, a minimal fix, and verification.

2026-03-2023

plan-fix

EffortlessMetrics/perl-lsp

Turn a discovery into a builder-ready handoff packet. Use when a scout or coordinator has identified a concrete slice and needs exact scope, verification, and routing rather than code changes.

2026-03-2023

scout-then-build

EffortlessMetrics/perl-lsp

Research-first workflow that scouts exact scope before launching a constrained builder. Use when a feature or fix is too unconstrained for a direct builder — converts ~50% success into ~90% by eliminating implicit decisions.

2026-03-2023

swarm-priorities

EffortlessMetrics/perl-lsp

Load roadmap weighting for scouts and coordinators so slices are ranked by actual repo priorities instead of convenience.

2026-03-2023

swarm-protocol

EffortlessMetrics/perl-lsp

Load shared swarm behavioral rules for coordinators and workers. Use for task routing, receipts, direct messaging, dedup, worktree boundaries, and lifecycle discipline.

2026-03-2023

Coding Standards

Banned In Production Code

unwrap(), expect() → use ?, .ok_or_else(), or pattern matching

panic!(), todo!(), unimplemented!() → return Result/Option

dbg!() → use tracing::debug!

std::process::abort() → never

std::process::exit() → only in bin/ directories and lifecycle.rs

Exception: one #[allow(clippy::expect_used)] in crates/perl-lsp/src/util/uri.rs

Patterns

Regex init: Option<Regex> with .ok() for graceful degradation

Non-empty collections: fixed-size arrays [T; N] for compile-time guarantees

.first() over .get(0)

.push(char) not .push_str("x") for single chars

or_default() not or_insert_with(Vec::new)

No unnecessary .clone() on Copy types

Test Standards

Return Result<()> from tests, or use perl_tdd_support::must / must_some

Descriptive names: test_<what>_<scenario>_<expected>

LSP tests: RUST_TEST_THREADS=2 cargo test -p perl-lsp -- --test-threads=2

Commit Format

Conventional: type(scope): description

Types: fix, feat, test, docs, chore, perf, refactor

Scope: crate name, subsystem, or repo area being changed

Verification

Run cargo fmt --all && cargo clippy -p <crate> --tests -- -D warnings && cargo test -p <crate>

Escalate to nix develop -c just ci-gate for changes spanning 3+ crates.

Git Commit Hygiene

Only git add files you intentionally created or modified

Never use git add -A or git add .

Exclude from commits unless your task requires them:

Cargo.lock unless dependencies changed
.claude/ infrastructure files unless that is the task
docs/project/CURRENT_STATUS.md
scripts/.ignored-baseline

Before committing, run git diff --cached --name-only

Dual Indexing

file_index.references.entry(bare_name.to_string()).or_default().push(symbol_ref.clone()); file_index.references.entry(qualified).or_default().push(symbol_ref);