Bitwarden Security Engineer
Chapter 39 of 59
5 minutes · 992 words · 8 sections
| Token | Label | When it applies |
|---|---|---|
| 🔴 | Update VULN Status | Child item has progressed (In Progress/Review) but VULN is still at a lower status |
| 🟡 | Mark Remediated | Child item is Done — set Remediation Date to merged PR date and move VULN to Remediated |
| 🟢 | Verify & Close | Fix is in a release that has already shipped — verify in prod, add Confirmation Date, close HackerOne |
| 🔵 | Monitor | Work is actively in progress or in a pending release; no action needed yet |
| ⚪ | Waiting | Child item exists but hasn’t started |
| ➖ | No Child Item | VULN is Ready for Resolution but no engineering ticket linked yet |
Use search_issues with this JQL:
project = VULN AND status not in (Done, Verified) AND "Source" = "HackerOne" ORDER BY priority DESC, updated DESCRequest fields: summary, status, description, priority, created, updated
Paginate if needed (default max 50; use nextPageToken to get all).
For each VULN key, run:
issue in linkedIssues("VULN-XXX")Request fields: summary, status, fixVersions, project
[VULN] in the summary are the primary engineering tracking items.Map Jira statuses to these categories:
| Category | Example statuses |
|---|---|
| Not Started | To Do, Backlog, Open, New, In Analysis |
| In Progress | In Progress, In Development, In Review, Code Review, In Testing |
| Done | Done, Closed, Resolved, Completed |
| Abandoned | Abandoned, Won’t Fix, Duplicate, Canceled |
For VULNs with multiple children: the highest-priority active child drives the action token. “In Progress” outranks “Not Started”; “Done” only counts if all non-abandoned children are Done.
JSON parsing rule — Always use gh‘s built-in --jq flag or standalone jq for JSON parsing. Never pipe to python3 or any other interpreter — Python is not in this skill’s allowed-tools and will trigger a permission prompt. If stderr suppression is needed, place 2>/dev/null after the full gh api ... --jq '...' command, not before:
# Correct — 2>/dev/null after --jq, before the next pipe
gh api --method GET "repos/bitwarden/REPO/compare/A...B?per_page=250" \
--jq '.commits[] | .commit.message | split("\n")[0]' 2>/dev/null \
| grep "#PR_NUMBER"PR search — Use gh search prs to find PRs. If that fails, fallback to the GitHub API with gh api --method GET "search/"
gh api --method GET "search/issues?q=CHILD-KEY+type:pr+org:bitwarden&per_page=10" \
--jq '.items[] | {number,title,state,mergedAt,url:.html_url}'For each PR that appears to be the correct fix (match on title/ticket key), get accurate merge details:
gh pr view PR_URL --json state,mergedAt,mergeCommit,baseRefName,titleDetermining release inclusion — Bitwarden’s repos (server, clients) use release branches with cherry-picks. The merge commit SHA on main gets a new SHA when cherry-picked, so compare/TAG...COMMIT_SHA always returns “diverged” and is unreliable. Do not use it.
The correct method is to compare consecutive release tags and search for the PR number in commit messages (cherry-picks preserve the original PR number):
# 1. List non-draft, non-prerelease tags for the relevant repo
gh release list --repo bitwarden/REPO --limit 20 \
--json tagName,publishedAt,isDraft,isPrerelease \
| jq '.[] | select(.isDraft == false and .isPrerelease == false)'
# 2. Find the two consecutive tags that bracket the expected fix release
# (e.g., v2026.4.0 and v2026.4.1)
# 3. List all commits in that range and grep for the PR number
gh api --method GET "repos/bitwarden/REPO/compare/TAG_PREV...TAG_RELEASE?per_page=250" \
--jq '.commits[] | .commit.message | split("\n")[0]' \
| grep "#PR_NUMBER"clients monorepo note: The bitwarden/clients repo publishes separate release tags per client type: web-vYYYY.M.P, cli-vYYYY.M.P, browser-vYYYY.M.P, desktop-vYYYY.M.P. A fix deployed in web-v2026.4.2 does not mean the browser extension has it — always check the specific product’s tag if the vulnerability affects a specific client.
Simple repos (e.g., sm-action) use direct pushes without cherry-picks. For those, compare/COMMIT_SHA...TAG returning "ahead" means the TAG is a descendant of the commit — i.e., the commit IS in the release.
To confirm a release has been deployed to production, check the published date from gh release list. If publishedAt is in the past and the release is not draft/prerelease, it is live.
Apply this decision tree to every VULN, using the child item statuses classified in Step 3 and the PR/release data from Step 4:
VULN status "Ready for Resolution":
→ No child items linked? → ➖ No Child Item
→ Child item exists, status Not Started? → ⚪ Waiting
→ Child item In Progress? → 🔴 Update VULN to In Progress
→ All child items Done? → 🟡 Mark Remediated
VULN status "In Progress" or "In Review":
→ Child item(s) still In Progress? → 🔵 Monitor
→ All child items Done, PR not yet found? → 🟡 Mark Remediated (investigate date)
→ All child items Done, PR merged? → 🟡 Mark Remediated (use PR merge date)
VULN status "Remediated":
→ Cannot determine release? → 🔵 Monitor
→ PR in an upcoming/unreleased version? → 🔵 Monitor (release pending)
→ PR in a released, deployed version? → 🟢 Verify & CloseThe Remediation Date should be the date the fix PR was merged to the default branch.
Use this template. Omit any section (including <details> blocks) that has zero items — do not render empty headings or empty tables.
# 🤖 HackerOne VULN Audit — {YYYY-MM-DD}
## Summary
| Token | Category | Count |
| ----- | ------------------------ | ----- |
| 🔴 | Need Status Update | {n} |
| 🟡 | Ready to Mark Remediated | {n} |
| 🟢 | Ready to Verify & Close | {n} |
| 🔵 | Monitoring | {n} |
| ⚪ | Waiting | {n} |
| ➖ | Missing Child Item | {n} |
{2–4 bullets: overall remediation health, anything overdue or stalled, patterns worth noting, any tickets with incomplete data that need manual follow-up}
## 🔴 Update VULN Status
| VULN | Priority | Summary | HackerOne | Child Item(s) | Child Status | Action |
| --------------- | -------- | ------------------------------ | --------------- | --------------- | ------------ | ------------------------ |
| [VULN-529](...) | High | Summary truncated to ~60 chars | [
Formatting notes:
[VULN-529](https://bitwarden.atlassian.net/browse/VULN-529)[#3673748](https://hackerone.com/reports/3673748). If not found, show unknown and flag it in the summary bullets.[PM-35250](https://bitwarden.atlassian.net/browse/PM-35250). If multiple, list each on its own line within the cell.[#1234](PR_URL) → v2026.8.0 ✅ deployed, [#1234](PR_URL) → v2026.9.0 ⏳ pending, or No PR foundInstall this repository
npx skills add bitwarden/ai-plugins/plugin marketplace add bitwarden/ai-pluginsSkills install per repository, not per chapter — the CLI has no documented per-skill form, so we do not print one.
main, last pushed 8 August 2026.SKILL.md, not by matching a directory convention. 13 distinct layouts observed: plugins/bitwarden-atlassian-tools/skills/*/SKILL.md, plugins/bitwarden-code-review/skills/*/SKILL.md, plugins/bitwarden-delivery-tools/skills/*/SKILL.md, plugins/bitwarden-design-tools/skills/*/SKILL.md, plugins/bitwarden-designer/skills/*/SKILL.md, plugins/bitwarden-devops-engineer/skills/*/SKILL.md, plugins/bitwarden-product-analyst/skills/*/SKILL.md, plugins/bitwarden-security-engineer/skills/*/SKILL.md, plugins/bitwarden-shepherd/skills/*/SKILL.md, plugins/bitwarden-tech-lead/skills/*/SKILL.md, plugins/bitwarden-testing-tools/skills/*/SKILL.md, plugins/claude-config-validator/skills/*/SKILL.md, plugins/claude-retrospective/skills/*/SKILL.md..claude-plugin/marketplace.json by Bitwarden, declaring 16 plugins. It is read for editorial metadata only — never as the skill index, which is always the repository tree./bitwarden/ai-plugins.md, and each chapter at its own .md URL.