Skip to content

fix: point drift-check at Infisical staging root path - #84

Merged
palmtr3man merged 1 commit into
mainfrom
fix/drift-check-infisical-root-path
Jul 9, 2026
Merged

fix: point drift-check at Infisical staging root path#84
palmtr3man merged 1 commit into
mainfrom
fix/drift-check-infisical-root-path

Conversation

@palmtr3man

@palmtr3man palmtr3man commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Updates scripts/drift-check.ts to read Infisical secrets from staging / instead of missing /tuj folder.
  • Pairs with newly configured GitHub Actions secrets (INFISICAL_TOKEN, NETLIFY_AUTH_TOKEN) and variable (NETLIFY_SITE_ID).

Test plan

  • Local npm run drift-check succeeds with configured credentials (warn mode reports expected parity gaps)
  • GitHub Actions drift-check workflow completes after merge

Made with Cursor

Summary by CodeRabbit

  • Bug Fixes
    • Corrected the Infisical path used during drift checks, so secret comparisons now pull from the expected location and report more accurate path details.

Secrets live under staging `/`, not `/tuj`. Unblocks guardrails after
GitHub Actions INFISICAL/Netlify credentials were configured.

Co-authored-by: Cursor <cursoragent@cursor.com>
@palmtr3man
palmtr3man merged commit f6a87a6 into main Jul 9, 2026
@haystack-code-reviewer-pr-hook

Copy link
Copy Markdown

Try Haystack Code Reviewer

Want AI-powered code review for this PR? Get instant analysis, interactive visualizations, and actionable insights.

Review this PR with Haystack

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f7f931c5-2efd-4580-9d06-043b23a55589

📥 Commits

Reviewing files that changed from the base of the PR and between 921f1c2 and d4d7ca6.

📒 Files selected for processing (1)
  • scripts/drift-check.ts

📝 Walkthrough

Walkthrough

The INFISICAL_PATH constant in scripts/drift-check.ts was changed from "/tuj" to "/", altering the secret path used for fetching Infisical keys and reporting drift.

Changes

Drift-check path update

Layer / File(s) Summary
Update Infisical secret path
scripts/drift-check.ts
Changes INFISICAL_PATH from "/tuj" to "/", affecting which Infisical secrets are fetched and reported.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/drift-check-infisical-root-path

Comment @coderabbitai help to get the list of available commands.

palmtr3man added a commit that referenced this pull request Sep 2, 2026
fix: address PR #84 alignment follow-ups (manifest reference, env validation, Netlify context preservation)
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.

1 participant