Skip to content

CI: R parse check and secret scan - #176

Open
Alvin-Nahabwe wants to merge 16 commits into
mainfrom
ci/app-checks
Open

CI: R parse check and secret scan#176
Alvin-Nahabwe wants to merge 16 commits into
mainfrom
ci/app-checks

Conversation

@Alvin-Nahabwe

Copy link
Copy Markdown
Contributor

What

Adds .github/workflows/ci.yml to the app repo:

Job What it does
r-parse Rscript scripts/check_parse.R syntax-checks every .R file (parse only — no packages). Catches breakage that otherwise only shows when the app launches.
secrets gitleaks history scan — advisory for now (continue-on-error).

Locally, all 105 R files parse cleanly, so r-parse is green on introduction.

Why advisory secrets

The repo has known secret-hygiene items in flight — the hardcoded dev cookie password in auth.R and the committed dev users.sqlite. Turning on a blocking gitleaks gate before those are cleared would fail every build. It runs advisory now; the follow-up secrets-hygiene PR flips it to blocking.

Follow-ups (separate PRs)

  • Contract check: validate that every field the R client sends to the DL API exists in the API's OpenAPI schema, so the drift class of bug (fixed in Align deep-learning client with the API contract #173) fails CI instead of production.
  • lintr style job once the tree is cleaned.

🤖 Generated with Claude Code

damiancodes and others added 16 commits July 3, 2026 00:29
  model training

  - Add all caret job queue label keys
Add async caret job queue with concurrent
.github/workflows/ci.yml runs on push to main and on PRs:

- r-parse: Rscript scripts/check_parse.R syntax-checks every .R file
  (parse only, no packages needed), catching breakage that would
  otherwise only appear when the Shiny app launches. All 105 current
  files parse.
- secrets: gitleaks history scan, advisory for now (continue-on-error).
  Known secret-hygiene items — the hardcoded dev cookie password and the
  committed dev users.sqlite — are being addressed separately; flip this
  to blocking once they are cleared.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants