Skip to content

Comments

Add documentation for secret configuration and integration test execution#41

Merged
ericwood73 merged 10 commits intomainfrom
copilot/refactor-schema-definition-process
Feb 16, 2026
Merged

Add documentation for secret configuration and integration test execution#41
ericwood73 merged 10 commits intomainfrom
copilot/refactor-schema-definition-process

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Integration tests require API keys but lacked documentation on environment setup and secret management in Copilot environments.

Changes

New: docs/TESTING_WITH_SECRETS.md

  • Copilot secret injection mechanism and configuration
  • Local development setup with .env file
  • Test execution commands with expected durations
  • Troubleshooting common errors (missing API key, network issues)
  • Environment variables reference

Updated: README.md

  • Quick start: install → configure → test
  • Test suite overview (unit vs integration)
  • Links to detailed documentation

Enhanced: .env.example

  • Header explaining security requirements
  • Required vs optional variables
  • Direct link to setup guide

Context

The schema simplification changes (removing JSON Schema executor) are complete and verified via static analysis. Integration tests cannot execute in current environment due to COPILOT_AGENT_INJECTED_SECRET_NAMES not exposing actual secret values, but documentation now provides clear path for verification with proper API key access.

Original prompt

So we are currently defining a schema simply as a set of fields during plan with a custom format and then using an execute node to convert that to a json schema. We did that originally to have a schema we could use for structured output. But given all of our state updates are determinisitic and we never output a full state do we even need that anymore?


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits January 30, 2026 23:13
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
…lign types

Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor schema definition and JSON conversion process Remove unnecessary JSON Schema conversion from schema extraction Jan 30, 2026
Copilot AI requested a review from ericwood73 January 30, 2026 23:24
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Copilot AI changed the title Remove unnecessary JSON Schema conversion from schema extraction Update spec processing tests for planner-only schema extraction Jan 30, 2026
Copilot AI and others added 2 commits January 31, 2026 23:54
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Copilot AI changed the title Update spec processing tests for planner-only schema extraction Remove unnecessary JSON Schema conversion from schema extraction Jan 31, 2026
Copilot AI changed the title Remove unnecessary JSON Schema conversion from schema extraction Remove JSON Schema conversion from schema extraction pipeline Feb 1, 2026
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Copilot AI changed the title Remove JSON Schema conversion from schema extraction pipeline Remove unnecessary JSON Schema conversion from schema extraction pipeline Feb 1, 2026
…ent setup

Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Co-authored-by: ericwood73 <2217397+ericwood73@users.noreply.github.com>
Copilot AI changed the title Remove unnecessary JSON Schema conversion from schema extraction pipeline Add documentation for secret configuration and integration test execution Feb 1, 2026
@ericwood73 ericwood73 merged commit 4188039 into main Feb 16, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Feb 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants