Skip to content

Change CUSTOM_CONFIG to use secrets#234

Open
mmuzhi wants to merge 3 commits intoTideDra:mainfrom
mmuzhi:main
Open

Change CUSTOM_CONFIG to use secrets#234
mmuzhi wants to merge 3 commits intoTideDra:mainfrom
mmuzhi:main

Conversation

@mmuzhi
Copy link
Copy Markdown

@mmuzhi mmuzhi commented Apr 11, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 11, 2026 17:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the scheduled GitHub Actions workflow to source CUSTOM_CONFIG from GitHub Secrets instead of GitHub Repository Variables, aiming to keep custom configuration out of the public/less-protected variables mechanism.

Changes:

  • Switch CUSTOM_CONFIG from ${{ vars.CUSTOM_CONFIG }} to ${{ secrets.CUSTOM_CONFIG }} in the main workflow.
  • Update the keep-alive timestamp file.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/main.yml Reads CUSTOM_CONFIG from GitHub Secrets instead of repository variables before generating config/custom.yaml.
.github/keep-alive.txt Updates the “Last run” timestamp used to keep scheduled workflows active.
Comments suppressed due to low confidence (1)

.github/workflows/main.yml:33

  • CUSTOM_CONFIG is now sourced from a GitHub Secret, but the workflow prints the resulting config/custom.yaml to logs via cat. Secret redaction is not guaranteed for multi-line/YAML content, so this can leak sensitive configuration. Avoid logging the file contents; if you need validation, log a checksum or only log in an explicitly enabled debug mode after masking.
          CUSTOM_CONFIG: ${{ secrets.CUSTOM_CONFIG }}
        run: |
          printf "%b\n" "$CUSTOM_CONFIG" > config/custom.yaml
          echo "Use custom config: "
          cat config/custom.yaml

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
OPENAI_API_BASE: ${{ secrets.OPENAI_API_BASE }}
CUSTOM_CONFIG: ${{ vars.CUSTOM_CONFIG }}
CUSTOM_CONFIG: ${{ secrets.CUSTOM_CONFIG }}
Copy link

Copilot AI Apr 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change switches CUSTOM_CONFIG from a repo variable to a secret, but other references/documentation appear to still describe CUSTOM_CONFIG as a public variable (and other workflows may still read vars.CUSTOM_CONFIG). Please update the remaining workflow(s)/docs to use the same source to prevent confusing setup instructions and failures when running alternative workflows.

Copilot uses AI. Check for mistakes.
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.

2 participants