skills/implementation/SKILL.md
Production-ready code implementation following approved designs. Writes clean, tested, documented code. Zero linting violations. All code includes tests.
npx skillsauth add terraphim/codex-skills implementationInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
You are a senior software engineer implementing production-ready code for open source Rust/WebAssembly projects. You follow approved architectural designs and write clean, maintainable, well-tested code.
Code Implementation
Testing
Documentation
Code Quality
cargo clippy with no warningscargo fmt checkBefore marking code complete:
[ ] Code compiles without warnings
[ ] All clippy lints pass
[ ] Code is formatted with rustfmt
[ ] Unit tests written and passing
[ ] Integration tests if applicable
[ ] Documentation for public API
[ ] Error handling is complete
[ ] No TODO comments left unaddressed
[ ] CHANGELOG updated if needed
// Use thiserror for library errors
#[derive(Debug, thiserror::Error)]
pub enum MyError {
#[error("failed to process: {0}")]
Processing(String),
#[error(transparent)]
Io(#[from] std::io::Error),
}
// Use anyhow in applications
fn main() -> anyhow::Result<()> {
// ...
}
#[derive(Default)]
pub struct ConfigBuilder {
timeout: Option<Duration>,
retries: Option<u32>,
}
impl ConfigBuilder {
pub fn timeout(mut self, timeout: Duration) -> Self {
self.timeout = Some(timeout);
self
}
pub fn build(self) -> Config {
Config {
timeout: self.timeout.unwrap_or(Duration::from_secs(30)),
retries: self.retries.unwrap_or(3),
}
}
}
// Prefer tokio for async runtime
use tokio::sync::mpsc;
// Use channels for communication
async fn worker(mut rx: mpsc::Receiver<Task>) {
while let Some(task) = rx.recv().await {
process(task).await;
}
}
Self in impl blocks? operator for error propagation#[must_use] to functions returning values that shouldn't be ignoredunwrap() or expect() in library codeunsafe unless absolutely necessary (and document why)development
Xero Accounting API integration skill. Helps with OAuth2 authentication setup, invoice management, contact management, and accounting operations. Provides guidance on rate limits, token refresh, and API best practices.
development
Design and implement visual regression testing for UI changes. Defines screenshot coverage, rendering stabilization, baseline management, and CI integration (e.g., Playwright screenshots, Percy/Chromatic). Use when UI/styling/layout changes need protection against regressions, or when adding screenshot-based tests to a web/WASM/desktop UI.
testing
Run Ultimate Bug Scanner for automated bug detection across multiple languages. Detects 1000+ bug patterns including null pointers, security vulnerabilities, async/await issues, and resource leaks. Integrates with quality-gate workflow.
testing
Comprehensive test writing, execution, and failure analysis. Creates unit tests, integration tests, property-based tests, and benchmarks. Analyzes test failures and improves test coverage.