Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions .github/workflows/verify.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name: verify

on:
push:
branches: [main]
pull_request:
branches: [main]

jobs:
verify:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: lts/*
cache: "npm"
- run: npm ci
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow uses npm ci on line 18, but since package-lock.json is listed in .gitignore (and thus not committed), npm ci will fail because it strictly requires a committed lockfile. Either change npm ci to npm install, or remove package-lock.json from .gitignore and commit the lockfile.

Suggested change
- run: npm ci
- run: npm install

Copilot uses AI. Check for mistakes.
- run: npm run verify
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
node_modules
.DS_Store
package-lock.json
CLAUDE.md
6 changes: 6 additions & 0 deletions .prettierrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{
"semi": true,
"singleQuote": false,
"tabWidth": 2,
"trailingComma": "es5"
}
5 changes: 5 additions & 0 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,11 @@ Source: github:metyatech/agent-rules@HEAD/rules/global/command-execution.md
- Do not assume agent platform capabilities beyond what is available; fail explicitly when unavailable.
- When building a CLI, follow standard conventions: --help/-h, --version/-V, stdin/stdout piping, --json output, --dry-run for mutations, deterministic exit codes, and JSON Schema config validation.

## Codex-only: Commands blocked by policy (PowerShell)

- `Remove-Item` (aliases: `rm`, `ri`, `del`, `erase`) → Use: `if ([IO.File]::Exists($p)) { [IO.File]::SetAttributes($p,[IO.FileAttributes]::Normal); [IO.File]::Delete($p) }`
- `Remove-Item -Recurse` (aliases: `rmdir`, `rd`) → Use: `if ([IO.Directory]::Exists($d)) { [IO.File]::SetAttributes($d,[IO.FileAttributes]::Normal); foreach ($e in [IO.Directory]::EnumerateFileSystemEntries($d,'*',[IO.SearchOption]::AllDirectories)) { [IO.File]::SetAttributes($e,[IO.FileAttributes]::Normal) }; [IO.Directory]::Delete($d,$true) }`

## Post-change deployment

After modifying code, check whether deployment steps beyond commit/push are needed before concluding.
Expand Down
25 changes: 25 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,31 @@ Returns APPROVE or FLAG with specific concerns.
npx skills add metyatech/skill-user-proxy --yes --global
```

## Development

### Prerequisites

- [Node.js](https://nodejs.org/) (LTS recommended)
- [npm](https://www.npmjs.com/)

### Setup

```bash
npm install
```

### Verification

```bash
npm run verify
```

### Formatting

```bash
npm run format
```

## License

MIT
1 change: 1 addition & 0 deletions index.js
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
// Agent skill entry point
20 changes: 20 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"name": "@metyatech/skill-user-proxy",
"version": "1.0.0",
"description": "Agent skill for user proxy review",
"type": "module",
"main": "index.js",
"scripts": {
"verify": "npm run format:check",
"format": "prettier --write .",
"format:check": "prettier --check ."
},
Comment on lines +7 to +11
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The verify script only runs format:check (Prettier). The AGENTS.md rules (line 125) require both formatting checks and linting to be enforced in CI, and line 126 states "Treat warnings as errors in CI." There is currently no linter configured for the JavaScript code (e.g., ESLint). Since the ruleset in this same repo mandates linting for every code repo, a linter should be added and included in the verify script.

Copilot uses AI. Check for mistakes.
"keywords": [
"agent-skill"
],
"author": "metyatech",
"license": "MIT",
"devDependencies": {
"prettier": "^3.0.0"
}
}
Loading