Latent debt surfaced by the reusable PHPCS job (previously continue-on-error).
PHPCS fixes (all pre-existing, not introduced by this PR):
- Add @package PeptideReconstitutionCalculator to file docblocks in all 12 PHP
files (previously missing; phpcbf cannot insert missing docblock tags).
- Replace two short-ternary ?: operators in class-prc-preset-provider.php with
full ternary (Universal.Operators.DisallowShortTernary.Found; lines 136+152).
JS lint fix:
- Add package.json + package-lock.json (no deps; lint script = node --check)
so the reusable lint-js job's npm ci / cache: npm steps succeed. The
calculator ships vanilla JS with no build toolchain — package.json is the
minimal stub the reusable workflow requires for has_js: true.
Agent-Session: claude/adopt-ci-20260616
P1 fix: Add class_exists() guard in PRC_Preset_Provider before
instantiating PR Core repository classes. If PR Core constant is
defined but classes fail to autoload, fall back to defaults instead
of crashing.
P2 fixes:
- Remove unused nonce from wp_localize_script (all endpoints public)
- Add cross-reference comments pinning SYRINGE_UNITS_PER_ML across
PRC_Math, calculator.js, and prcConfig.syringeUnits
QA verdict: qa-reviews/peptide-reconstitution-calculator/2026-04-17-9aa5309.md
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
