Agent skill

Test Conversion Reviewer

by nwjs in nwjs/chromium.src

Reviewer for converting unit tests to browser tests for Project Bedrock.

BSD-3-ClauseAuto-check passedTesting & QA

Install Test Conversion Reviewer

skills CLI
$ npx skills add nwjs/chromium.src --skill test-conversion-reviewer -a claude-code

Project install by default; add -g for ~/.claude/skills/.

GitHub CLI
$ gh skill install nwjs/chromium.src test-conversion-reviewer --agent claude-code

Project scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).

Manual copy
$ git clone --depth 1 https://github.com/nwjs/chromium.src.git skills-src && mkdir -p .claude/skills && cp -r skills-src/agents/projects/bedrock/test-conversion-reviewer .claude/skills/test-conversion-reviewer && rm -rf skills-src

Use ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.

Claude Code skills documentation · loads skills from .claude/skills/

Facts

Skill name
test-conversion-reviewer
GitHub stars
160
Token cost
~776 tokens
SKILL.md length
425 words
Files
1
Skills in repo
64
Repo updated
First seen
Licence
BSD-3-Clause

At a glance

Reviewer for converting unit tests to browser tests for Project Bedrock.

  • Works in 2 steps: A parent commit performing a pure… → A child commit containing the actual…
  • Needs to review their work while removing complex Browser dependencies from tests
  • SKILL.md covers Your Job, Split Rename Verification…, Output Format and Important
  • Instructions only: no scripts, shell commands, URLs or credentials in SKILL.md

What it does

Test Conversion Reviewer is an agent skill from nwjs/chromium.src. Reviewer for converting unit tests to browser tests for Project Bedrock. Invoke this when the user needs to review their work while removing complex Browser dependencies from tests.

Its SKILL.md is about 780 tokens, which your agent loads only when the skill is triggered. It is a single SKILL.md file with no bundled scripts.

It sits in Testing & QA, covering Unit testing. The repository describes itself as: Chromium codebase with NW.js modifications. Based on https://chromium.googlesource.com/chromium/src.git. The licence is BSD-3-Clause.

When your agent uses it

  • Needs to review their work while removing complex Browser dependencies from tests
  • Tasks that involve Unit testing

Example prompts

  • “/test-conversion-reviewer”

Workflow steps

2 steps, taken from the first numbered list in SKILL.md.

  1. A parent commit performing a pure rename/move of the target test file
  2. A child commit containing the actual functional changes and browser test

What it can do on your machine

Read from SKILL.md and the folder at commit a9e8946. It shows what the files ask for, not the result of running them.

  • Tool permissions

    Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.

    From allowed-tools in the SKILL.md frontmatter.

  • Runs code

    No scripts in the folder and no shell commands in SKILL.md.

    From the folder's file list and the shell code blocks in SKILL.md.

  • Network

    No URLs in SKILL.md.

    From URLs in SKILL.md, links to its own repository left out.

  • Credentials

    Names no API keys, tokens, secrets or passwords.

    From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.

Context cost

Test Conversion Reviewer loads about 776 tokens when it runs. Until then it costs about 52 tokens; SKILL.md has 425 words of instructions outside code blocks.

Always · name and description, kept in context so the agent knows when to use it
~52
When it runs · the whole SKILL.md, loaded when a task matches
~776

Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.

Safety

Auto-check passed

The automated check found no risky patterns in SKILL.md.

Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.

SKILL.md

The full file from nwjs/chromium.src at commit a9e8946, republished under its BSD-3-Clause licence (© nwjs). 425 words, ~776 tokens.

Download SKILL.mdSave it as .claude/skills/test-conversion-reviewer/SKILL.md (or your agent's skills folder).
name
test-conversion-reviewer
description
Reviewer for converting unit tests to browser tests for Project Bedrock. Invoke this when the user needs to review their work while removing complex Browser dependencies from tests.

Role

You are the Reviewer for Project Bedrock — eliminating Browser* dependencies from tests.

Your Job

Review CLs that eliminates Browser* dependency from a unit test. You must use jj log and jj diff to inspect the commit history and diffs directly. Please review the CL to ensure that each commit is focused, easy to review, and well-scoped with no unnecessary changes.

Make sure to look for unnecessary overrides such as:

void SetUpOnMainThread() override {
InProcessBrowserTest::SetUpOnMainThread();
}

Reject CLs that still include an EXECUTION_PLAN.md

Reject CLs that contain stray tool-generated metadata, internal config files, or directories in the active changes/working copy. Ensure that all changes are strictly scoped to the test migration and its clean implementation.

Reject any commits that include TAG= or CONV= in their commit description or message. Ensure that these tags are completely absent from the uploaded commit metadata.

Make sure comments were not removed if the attached code remains unchanged.

Make sure no tests were disabled by this CL. Verify file contents via view_file if you suspect tests were deleted, rather than relying solely on diff snippets, to avoid confusion caused by truncated diffs.

Accept Browser* dependencies (like browser()) if the test has been explicitly converted to a browser test (Fallback Pattern).

Make sure the SetUp/TearDown methods don't call testing::Test::SetUp() or testing::Test::TearDown(), since those are trivial.

Lastly, it's valuable to minimize the size of a diff (e.g. introducing a profile() getter if the tests all use one, rather than converting all the accesses to profile_). Look for opportunities to make the CL easier to review.

Show full SKILL.md (173 more words)Show less

Split Rename Verification (Jujutsu relation chain)

Reviewers heavily value clean diffs where renames are isolated from functional changes. If a task involves moving or renaming a test file (e.g., moving a .cc to a _browsertest.cc or similar target change):

  • Inspect jj log and ensure the commit history is split into two clean, sequential changes:
  1. A parent commit performing a pure rename/move of the target test file (with 0 code modifications to the test content).
  2. A child commit containing the actual functional changes and browser test setup on top.
  • Reject the CL if these changes are squashed into a single commit. Guide the developer agent to split the change using jj.

Output Format

Output ONLY a list of blocking comments. Each must be a concrete, actionable issue with the file name and what needs to change. Do NOT include non-blocking observations, style nits, or cosmetic issues. Do NOT include issues in unrelated targets.

If there are no blocking issues, output exactly: LGTM

Important

  • You are READ-ONLY during review. Do not modify files.

© nwjs, BSD-3-Clause. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file

Files

Just SKILL.md in agents/projects/bedrock/test-conversion-reviewer of nwjs/chromium.src.

Open the folder on GitHubat commit a9e8946

Compare with similar skills

Test Conversion Reviewer next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.

Test Conversion Reviewer compared with similar skills
SkillStarsUsed inTokensAuto-checkLicenceRepo updated
Test Conversion Reviewer this skillnwjs/chromium.src160—~776Automated safety check: PassBSD-3-Clause
TDD WorkflowhellangleZ/burn-in-cceverywhere-ralph11211 repos~2.4kAutomated safety check: PassNone
Testing OpenLogi UIAprilNEA/OpenLogi23k—~1.1kAutomated safety check: PassApache-2.0
Go Testingcxuu/golang-skills1701 repos~1.3kAutomated safety check: PassApache-2.0
Contractssamchon/nestia2.2k—~1.3kAutomated safety check: PassMIT
Cohesion Over TestabilityEpicenterHQ/epicenter4.8k—~2kAutomated safety check: PassCustom licence

Similar skills

  • TDD Workflow

    hellangleZ/burn-in-cceverywhere-ralph

    A skill your agent uses when writing new features, fixing bugs, or refactoring code.

    112 GitHub starsUsed in 11 repos~2.4k tokens
    Testing & QAAuto-check passed
  • Testing OpenLogi UI

    AprilNEA/OpenLogi

    Verifies OpenLogi's native GPUI interface with focused tests, the component gallery and a mock agent, choosing the evidence that fits each change.

    23k GitHub stars~1.1k tokensUpdated 5 days ago
    Testing & QAAuto-check passed
  • Go Testing

    cxuu/golang-skills

    A skill your agent uses when writing, reviewing, or improving Go test code — including table-driven tests, subtests, parallel tests, test helpers, test doubles, and assertions with cmp.Diff.

    170 GitHub starsUsed in 1 repo~1.3k tokens
    Testing & QAAuto-check passed
  • Contracts

    samchon/nestia

    Defines self-acknowledgments for production declarations and tests.

    2.2k GitHub stars~1.3k tokensUpdated yesterday
    Testing & QAAuto-check passed
  • Cohesion Over Testability

    EpicenterHQ/epicenter

    Collapse test-shaped production boundaries while preserving behavior and coverage.

    4.8k GitHub stars~2k tokensUpdated yesterday
    Testing & QAAuto-check passed
  • JS-in-HTML Testing

    liaohch3/claude-tap

    Tests JavaScript embedded in an HTML file in two layers: pytest checks of the logic ported to Python, and Playwright runs in a real browser for the DOM.

    3.3k GitHub stars~924 tokensUpdated 16 days ago
    Testing & QAAuto-check passed

More from nwjs/chromium.src

All 64 skills in this repo
  • Analyzing SQL Traces

    nwjs/chromium.src

    Extracts raw trace data from Perfetto traces, runs arbitrary SQL queries for custom follow-up analysis, and applies expert cognitive principles (Tiered Flow Analysis, Semantic Mismatch, Redundancy)…

    160 GitHub stars~2.9k tokensUpdated 5 days ago
    Auto-check passed
  • Autonomous multi-agent performance optimization loop for Chromium and V8.

    160 GitHub stars~4.2k tokensUpdated 5 days ago
    Auto-check passed
  • Automated Tracing

    nwjs/chromium.src

    Automated Tracing & Performance Telemetry in Chromium using Perfetto and Telemetry benchmarks.

    160 GitHub stars~1.5k tokensUpdated 5 days ago
    Auto-check passed
  • Chrome Releases

    nwjs/chromium.src

    Queries Chrome commit, version, release, and milestone metadata.

    160 GitHub stars~1.3k tokensUpdated 5 days ago
    Auto-check passed
  • Chromium Docs

    nwjs/chromium.src

    Search and reference Chromium documentation from the local docs index, including design docs, APIs, and development guides.

    160 GitHub stars~1.2k tokensUpdated 5 days ago
    Auto-check passed
  • Gn Deps Debugging

    nwjs/chromium.src

    Diagnose Chromium GN dependency and include-visibility failures, including BUILD.gn deps/publicdeps, DEPS include rules, private headers, and circular dependencies.

    160 GitHub stars~1.5k tokensUpdated 5 days ago
    Auto-check passed

Categories

Questions about Test Conversion Reviewer

What does Test Conversion Reviewer do?

Reviewer for converting unit tests to browser tests for Project Bedrock. src. Reviewer for converting unit tests to browser tests for Project Bedrock.

When should I use Test Conversion Reviewer?

Test Conversion Reviewer fits situations like: needs to review their work while removing complex Browser dependencies from tests; tasks that involve Unit testing.

How do I install Test Conversion Reviewer in Claude Code?

Run `npx skills add nwjs/chromium.src --skill test-conversion-reviewer -a claude-code`. Or copy the skill folder (agents/projects/bedrock/test-conversion-reviewer in nwjs/chromium.src) into .claude/skills/test-conversion-reviewer in your project. Claude Code loads it when a task matches its description.

How do I install Test Conversion Reviewer in Codex?

Run `npx skills add nwjs/chromium.src --skill test-conversion-reviewer -a codex`. Or copy the skill folder (agents/projects/bedrock/test-conversion-reviewer in nwjs/chromium.src) into .agents/skills/test-conversion-reviewer in your project. Codex loads it when a task matches its description.

Can I use Test Conversion Reviewer in Cursor, Gemini CLI or GitHub Copilot?

Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add nwjs/chromium.src --skill test-conversion-reviewer -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/test-conversion-reviewer, .gemini/skills/test-conversion-reviewer, .github/skills/test-conversion-reviewer and .opencode/skills/test-conversion-reviewer in your project.

What does Test Conversion Reviewer need to run?

SKILL.md names no scripts, command-line tools or credentials: Test Conversion Reviewer is instructions for the agent only.

Does Test Conversion Reviewer access the network?

SKILL.md contains no URLs. Any network use would come from the scripts or tools the agent runs. This is read from the text; nothing was executed.

Is Test Conversion Reviewer safe to install?

Our automated static check of SKILL.md found no risky patterns, such as piping downloads into a shell, reading credential files or hidden Unicode. It is not a guarantee. Review the folder before installing.

What licence does Test Conversion Reviewer use?

Test Conversion Reviewer is published under the BSD-3-Clause licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.

How many tokens does Test Conversion Reviewer use?

About 776 tokens (SKILL.md is roughly 3.1k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.

What are the alternatives to Test Conversion Reviewer?

Skills that share tags, products or a category with Test Conversion Reviewer: TDD Workflow (hellangleZ/burn-in-cceverywhere-ralph, 112 stars), Testing OpenLogi UI (AprilNEA/OpenLogi, 23k stars), Go Testing (cxuu/golang-skills, 170 stars) and Contracts (samchon/nestia, 2.2k stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.

Who maintains Test Conversion Reviewer?

nwjs (a GitHub organization) maintains it in nwjs/chromium.src, which has 160 GitHub stars. The repository holds 64 skills in this directory. The repository was last updated on October 3, 2026.

Source: nwjs/chromium.src on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.