From 91fb21ba6736f0b8808b95a53e56db2333f1f721 Mon Sep 17 00:00:00 2001 From: Ahmad Wilson Date: Fri, 19 Jun 2026 13:32:41 -0500 Subject: [PATCH 1/2] fix(ci): use dummy key fallback so dependabot PRs pass test_parse_aws GitHub withholds secrets from dependabot PRs, so OPENAI_API_KEY arrived as an empty string. The parser's non-empty guard then short-circuited before the mocked requests.post was ever called, causing test_parse_aws to fail with 0 subcommands instead of 2. Using a literal fallback lets the parser proceed while the test mock handles the actual HTTP call. Co-authored-by: Cursor --- .github/workflows/CI.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 7fedf70..9a7989c 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -27,6 +27,8 @@ jobs: - name: Run tests env: - OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }} + # Fallback ensures the parser's non-empty-key guard doesn't short-circuit + # in dependabot PRs where GitHub withholds secrets. The real API is mocked. + OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY || 'dummy-key-for-ci' }} run: | PYTHONPATH=src pytest tests/test_parser.py --verbose From 6b34a75114353a7560d866740c63e5dffe2a1e9d Mon Sep 17 00:00:00 2001 From: Ahmad Wilson Date: Fri, 19 Jun 2026 14:04:03 -0500 Subject: [PATCH 2/2] fix(ci): bump python-version to 3.12 to support requests>=2.34.2 requests 2.34.x requires Python >=3.10. The CI was still pinned to 3.9, causing pip to reject the version outright. Using 3.12 aligns with the Dockerfile (3.14-slim bump in PR #44) and unblocks #42. Co-authored-by: Cursor --- .github/workflows/CI.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 9a7989c..991c933 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -17,7 +17,7 @@ jobs: - name: Set up Python uses: actions/setup-python@v4 with: - python-version: "3.9" + python-version: "3.12" - name: Install dependencies run: |