← Back to scan
File as GitHub Issue repo: PolicyEngine/policyengine-us

Push this scan report to PolicyEngine/policyengine-us

Click the green button below to open GitHub’s new-issue form, pre-filled with the report title, summary table, top findings, and an embedded score-card image. No authentication needed — you review on GitHub before submitting. Repobility is credited as the scanner.

Embedded score card image

This image will render at the top of the issue body. Hosted on Repobility, refreshes automatically after re-scans.

Repobility score card

Issue title

`self.update_variable` used but never assigned in __init__

Curate findings to include

Pick exactly which findings appear in the issue body. By default the top 5 are included. Uncheck noise, check what matters.

Top 5 (default)
Severity Rule Title File:line
CRIT MINED107 [MINED107] Missing import: `html` used but not imported: The file uses `html.something(..… policyengine_us/tools/add_plotly_to_boo…:21
CRIT MINED125 [MINED125] GHA script injection via github.head_ref in run-step: Multi-line `run: |` bloc… .github/workflows/pr.yaml:58
CRIT MINED116 [MINED116] Workflow uses `secrets.CODECOV_TOKEN` on a `pull_request` trigger: This workfl… .github/workflows/pr.yaml:169
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/taxsim.py:131
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/taxsim.py:130
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/taxsim.py:129
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/taxsim.py:128
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/dc_kccatc.py:80
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/winship.py:76
HIGH MINED108 [MINED108] `self.update_variable` used but never assigned in __init__: Method `apply` of … policyengine_us/reforms/dc_tax_threshol…:30
HIGH MINED108 [MINED108] `self.branched_simulation` used but never assigned in __init__: Method `__exit… policyengine_us/tools/branched_simulati…:29
HIGH MINED108 [MINED108] `self.computed_variables` used but never assigned in __init__: Method `__exit_… policyengine_us/tools/branched_simulati…:30
HIGH MINED108 [MINED108] `self.branched_simulation` used but never assigned in __init__: Method `__ente… policyengine_us/tools/branched_simulati…:26
HIGH MINED108 [MINED108] `self.branched_simulation` used but never assigned in __init__: Method `__ente… policyengine_us/tools/branched_simulati…:24
HIGH MINED108 [MINED108] `self.computed_variables` used but never assigned in __init__: Method `__enter… policyengine_us/tools/branched_simulati…:22
HIGH MINED108 [MINED108] `self.get_year` used but never assigned in __init__: Method `__getitem__` of c… policyengine_us/data/dataset_schema.py:240
HIGH MINED108 [MINED108] `self.years` used but never assigned in __init__: Method `label` of class `USM… policyengine_us/data/dataset_schema.py:220
HIGH MINED108 [MINED108] `self.years` used but never assigned in __init__: Method `name` of class `USMu… policyengine_us/data/dataset_schema.py:215
HIGH MINED001 [MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e… policyengine_us/variables/gov/ssa/reven…:51
HIGH MINED001 [MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e… policyengine_us/variables/gov/ssa/reven…:51
HIGH MINED001 [MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e… policyengine_us/variables/gov/puf.py:54
HIGH SEC029 [SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT… policyengine_us/tools/taxsim/generate_t…:122
HIGH SEC078 [SEC078] Python: requests without timeout: requests.get/post without a timeout will hang … policyengine_us/tools/taxsim/generate_t…:122
HIGH SEC078 [SEC078] Python: requests without timeout: requests.get/post without a timeout will hang … policyengine_us/tools/geography/downloa…:127
HIGH MINED009 [MINED009] Floats For Money: Variable named price/amount/cost typed as float instead of D… policyengine_us/reforms/ssa/trustees_co…:29
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … policyengine_us/reforms/ssa/trustees_co…:114
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … policyengine_us/reforms/ssa/trustees_20…:139
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … policyengine_us/build_metadata.py:65
HIGH SEC103 [SEC103] LDAP injection — non-constant search filter: User input concatenated into an LDA… .github/bump_version.py:10
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:283
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:279
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:277
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:261
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:257
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:255
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:225
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:221
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:219
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:203
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:199
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:197
HIGH MINED115 [MINED115] Action `codecov/codecov-action` pinned to mutable ref `@v6`: `uses: codecov/co… .github/workflows/pr.yaml:163
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:118
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:114
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:108
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:94
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu… .github/workflows/pr.yaml:90
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:88
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v5`: `uses: astral-sh/setu… .github/workflows/pr.yaml:71
HIGH MINED115 [MINED115] Action `actions/setup-python` pinned to mutable ref `@v5`: `uses: actions/setu… .github/workflows/pr.yaml:67
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout… .github/workflows/pr.yaml:65
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:24
HIGH MINED115 [MINED115] Action `astral-sh/setup-uv` pinned to mutable ref `@v8.1.0`: `uses: astral-sh/… .github/workflows/pr.yaml:15
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout… .github/workflows/pr.yaml:13
MED MINED109 [MINED109] Mutable default argument in `variation_chart` (list): `def variation_chart(...… policyengine_us/tools/documentation.py:54
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… policyengine_us/build_metadata.py:54
MED ERR001 [ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even… policyengine_us/variables/gov/ssa/reven…:51
MED ERR001 [ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even… policyengine_us/variables/gov/ssa/reven…:51
MED ERR001 [ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even… policyengine_us/variables/gov/puf.py:54
MED SEC012 [SEC012] ZipSlip — Archive Path Traversal: Archive extraction without path validation all… policyengine_us/tools/geography/downloa…:169
MED SEC136 [SEC136] AI-typical over-broad exception handler swallowing all errors: Catch-all excepti… policyengine_us/build_metadata.py:48
MED COMP001 [COMP001] High cognitive complexity: Function `__init__` has cognitive complexity 20 (Son… policyengine_us/data/dataset_schema.py:61
MED COMP001 [COMP001] High cognitive complexity: Function `__init__` has cognitive complexity 24 (Son… policyengine_us/data/dataset_schema.py:173
LOW COMP001 [COMP001] High cognitive complexity: Function `_iter_surface_files` has cognitive complex… policyengine_us/build_metadata.py:22
LOW AIC003 Duplicated implementation block across source files policyengine_us/variables/gov/hud/pha_p…:15
LOW AIC003 Duplicated implementation block across source files policyengine_us/variables/gov/hhs/medic…:30
LOW AIC003 Duplicated implementation block across source files policyengine_us/variables/gov/aca/slspc…:28
LOW AIC003 Duplicated implementation block across source files policyengine_us/tools/geography/downloa…:68
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/taxsim.py:36
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/states/nc/eitc/…:24
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/harris/capital_…:3
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/ctc/ctc_older_c…:3
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/crfb/tax_employ…:5
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/crfb/tax_employ…:5
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/congress/tlaib/…:138
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/congress/tlaib/…:66
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/congress/tlaib/…:94
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/aca/aca_ptc_sim…:32
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/aca/aca_ptc_sim…:28
LOW AIC003 Duplicated implementation block across source files policyengine_us/reforms/aca/aca_ptc_add…:31
LOW AIC002 Source file name looks like an AI patch artifact policyengine_us/variables/gov/states/mt…:1
LOW AIC002 Source file name looks like an AI patch artifact policyengine_us/variables/gov/states/dc…:1
INFO MINED050 [MINED050] Stub Only Function: Function declared but body is just pass, return None, rais… policyengine_us/variables/gov/ssa/reven…:52
INFO MINED050 [MINED050] Stub Only Function: Function declared but body is just pass, return None, rais… policyengine_us/variables/gov/ssa/reven…:52
INFO MINED050 [MINED050] Stub Only Function: Function declared but body is just pass, return None, rais… policyengine_us/variables/gov/puf.py:55
INFO MINED043 [MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr… policyengine_us/variables/gov/local/ca/…:11
INFO MINED043 [MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr… policyengine_us/variables/gov/local/ca/…:11
INFO MINED043 [MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr… policyengine_us/tools/taxsim/generate_t…:25
INFO MINED067 [MINED067] Python Requests No Timeout: requests.get/post/etc. without timeout= can hang f… policyengine_us/tools/taxsim/generate_t…:122
INFO MINED067 [MINED067] Python Requests No Timeout: requests.get/post/etc. without timeout= can hang f… policyengine_us/tools/geography/downloa…:127
INFO MINED067 [MINED067] Python Requests No Timeout: requests.get/post/etc. without timeout= can hang f… policyengine_us/tools/download_hud_fmr.…:40
INFO MINED049 [MINED049] Print Pii: Logging password/token/email/ssn directly to stdout. policyengine_us/tools/download_hud_fmr.…:101
Reset to top 5 92 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `PolicyEngine/policyengine-us`

**Score: 79/100 (B)**  ·  130 findings  ·  scanned 2026-06-05 22:34 UTC  ·  179,713 LOC

| Severity | Count |
|---|---|
| CRITICAL | 3 |
| HIGH | 51 |
| MEDIUM | 9 |
| LOW | 19 |

📊 [Full filterable report](https://repobility.com/scan/79dd3e5f-0efc-478f-8e6a-2752562fbadd/)  ·  ![scorecard](https://repobility.com/scan/79dd3e5f-0efc-478f-8e6a-2752562fbadd/report.png?v=1780698875-s2)

### Top findings

1. **CRITICAL** `MINED107` — Missing import: `html` used but not imported
   `policyengine_us/tools/add_plotly_to_book.py:21` · ✓ Repobility
2. **CRITICAL** `MINED125` — GHA script injection via github.head_ref in run-step: Multi-line `run
   `.github/workflows/pr.yaml:58` · ✓ Repobility
3. **CRITICAL** `MINED116` — Workflow uses `secrets.CODECOV_TOKEN` on a `pull_request` trigger
   `.github/workflows/pr.yaml:169` · ✓ Repobility
4. **HIGH** `MINED108` — `self.update_variable` used but never assigned in __init__
   `policyengine_us/reforms/taxsim.py:131` · ✓ Repobility
5. **HIGH** `MINED108` — `self.update_variable` used but never assigned in __init__
   `policyengine_us/reforms/taxsim.py:130` · ✓ Repobility

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/79dd3e5f-0efc-478f-8e6a-2752562fbadd/_
Megaproject â high spam risk
Could not determine 'PolicyEngine/policyengine-us' star count (GitHub API rate-limited or unreachable). When in doubt about repo size, prefer opening a focused PR or a discussion rather than an issue.
Already filed
70/136 findings (51%) on this scan are already flagged as test-file, won't-fix, or suppressed. The scan is too noisy to file as a single issue. Curate down to specific actionable findings, or address the FP source first.
I read the warnings â override Cancel

The button opens GitHubâs new-issue page in a new tab. You will see the title + body pre-filled â review, edit if you want, then click GitHubâs "Submit new issue" button. Repobility never posts anything on your behalf.

For real security findings on big repos: use the project's SECURITY.md or private advisory flow instead of a public issue.