Security audit

golang-refactoring

Security checks for vulnerabilities and agentic risk

Overview

This Go refactoring skill is purpose-aligned and clearly structured around human-approved, test-backed code changes, with only ordinary tooling supply-chain cautions.

Install this if you want an agent to help with Go refactoring and are comfortable granting it repository editing, Go tool execution, git/gh workflow, and sub-agent worktree authority. In higher-assurance environments, pin the Go tool versions instead of using @latest and review every generated diff before merging.

A.I.G

Vulnerability Patterns

Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
Insecure DependenciesIntroduces malicious components through unsafe dependency sources
Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally

Findings (2)

T08 · Insecure Dependencies

Warning

Location: SKILL.md:17
Finding: Unpinned Go Tool Dependencies Resolve Mutable Upstream Releases
Content: View full analysis
Remediation: View remediation

T03 · Remote Payload Retrieval and Execution

Warning

Location: references/go-tooling.md:87
Finding: Remote Go Program Is Downloaded and Immediately Executed Using a Mutable Version
Content: View full analysis
Remediation: View remediation

SkillSpector

Vulnerability Patterns

Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
Supply ChainUnpinned Dependencies, External Script Fetching, Obfuscated Code
Excessive AgencyUnrestricted Tool Access, Autonomous Decision Making, Scope Creep

Findings (1)

Anti-Refusal Statement

High

Category: Anti-Refusal
Confidence: 80% confidence
Finding: Skill instructs the agent to omit warnings, disclaimers, or ethical commentary. Stripping safety caveats hides risk from the user and is a common jailbreak preamble.
Content: Scanner excerpt · references/structural.md (reported line 129)May include surrounding context.
md
- **Deprecate before deleting.** A doc comment beginning `// Deprecated: ...` is recognized by tooling and IDEs and surfaces as a strikethrough or warning at every call site, which gives callers time to migrate before the symbol disappears — deleting an exported identifier outright breaks every downstream module at their next `go build` with no warning beforehand.

Static analysis

No suspicious patterns detected.