-
Notifications
You must be signed in to change notification settings - Fork 263
fix: upgrade workflows, example modules, and test harnesses #2277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+1,657
−901
Merged
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
bbc90ca
fix: upgrade workflows, example modules, and test harnesses
matttrach f8d2a11
fix: add new command to make phony
matttrach b9887c6
fix: add agentic programming structure and updating tests
matttrach 0223938
fix: use external check maintainer, format tests, use real password f…
matttrach e023f33
fix: correct the go version
matttrach 4d6a7bf
fix: update readme to correct go version
matttrach 6d4050c
fix: contents read permission necessary for checkout
matttrach 3212be9
fix: remove set x in run tests
matttrach 84347cd
fix: separate out file path properly
matttrach d2f24af
fix: contents read permission necessary for checkout
matttrach 0368377
fix: remove set e on create rc tag script
matttrach 5111ba2
fix: validate number with good error message
matttrach fe96572
fix: validate pr number with good error message
matttrach bb90b27
fix: all github rest calls should paginate
matttrach 6f253ac
fix: rerun tests needs dirty mode
matttrach ab63acf
fix: set script to standard module exports
matttrach 097ce51
fix: use test context over a new background context
matttrach 0dfd731
fix: use bash to prevent dash from breaking things
matttrach bf82163
fix: use standard export default over module exports
matttrach 1c5b88c
fix: add repo links to fossa workflow
matttrach 7babe3f
fix: fix typo
matttrach f48c102
fix: remove read all for least privilege
matttrach 101ab69
fix: test compile checks target packages instead of files
matttrach fd21401
fix: use the proper import
matttrach d3e57a0
fix: prevent null latest rc number from crashing tag script
matttrach b84b33d
fix: use import with the local file url
matttrach e449ff8
fix: prevent malformed tags due to missing tag
matttrach 2f0c120
fix: remove err from cleanup
matttrach 6d0511a
fix: remove setup-go
matttrach 450f83c
fix: use a different file suffix to indicate an ESM file
matttrach 495ae0e
fix: add links as necessary and check out earlier in manual release
matttrach 876f69a
fix: move checkout up the workflow
matttrach f9697f9
fix: update the readme
matttrach ea56789
fix: move checkout to top of job
matttrach 8711968
fix: move checkout to top of job
matttrach 16119b8
fix: prevent crash by validating key pair output length
matttrach 751f7f1
fix: validate wait time
matttrach 192670d
fix: replace base64 with hex identifier
matttrach 0055b50
fix: run compile check with dash r
matttrach 9022c84
fix: use production lets encrypt
matttrach 3a1ed91
fix: wrap error
matttrach 09130d2
fix: wrap errors
matttrach f21ead3
fix: check for nil
matttrach 85c6a8a
fix: remove failed tests report after retrieving the data
matttrach f008aaa
fix: improve missing key pair error
matttrach c3dbd4a
fix: add nix install to build test
matttrach 9de070c
fix: node lint
matttrach 8f27ce4
fix: spacing
matttrach bdfc87d
fix: fetch full depth and validate git version
matttrach 2e0f4ed
fix: add issues write permission
matttrach 9b3d428
fix: add issues write permission
matttrach b0a94e0
fix: validate issue body
matttrach 47b1245
fix: add build to tests
matttrach 4883583
fix: spacing
matttrach 5306852
fix: move the commit validation script to external
matttrach 89d3def
fix: fall back to commit log and pass pr number through nix
matttrach 0ff49a5
fix: revert changes to product
matttrach fb3ee6b
fix: no truncated messages
matttrach e4b2ac7
fix: add repo to word list
matttrach c14238c
fix: add url to words
matttrach 6358418
fix: interpret terraform formatting issues
matttrach d897a5b
fix: make sure to fail if terraform format fails
matttrach File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # AI Agent Memory | ||
|
|
||
| This directory contains persistent context and learnings to retain across AI sessions. | ||
|
|
||
| Agents can read from and write to this directory to remember past decisions, project-specific quirks, and user preferences, improving future interactions. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # Specialized AI Agents | ||
|
|
||
| This directory contains specialized agent definitions and prompts. | ||
|
|
||
| These files can be used to set the context, persona, and specific capabilities for different AI agents operating within the repository. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # AI Output Styles | ||
|
|
||
| This directory contains guidelines on how AI agents should format their responses. | ||
|
|
||
| Rules here ensure that code suggestions, pull request reviews, and conversational assistance maintain a consistent and readable structure. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| # Claude Strict Output Style | ||
|
|
||
| As an agentic programming assistant, your output must be strictly utilitarian: | ||
|
|
||
| ## 1. No Conversational Filler | ||
| - Skip all greetings, pleasantries, and conversational transitions. | ||
| - Output only the requested code, diffs, or execution results. | ||
|
|
||
| ## 2. Format Requirements | ||
| - Provide file modifications as standard unified diffs or complete code blocks. | ||
| - Use absolute file paths for any file references. | ||
|
|
||
| ## 3. Explanations | ||
| - Provide explanations ONLY if explicitly requested by the user. | ||
| - If an error occurs, output the error message in a bold blockquote at the very top. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| # Gemini Conversational Style | ||
|
|
||
| As a coding partner, your output should be collaborative, educational, and conversational: | ||
|
|
||
| ## 1. Conversational Tone | ||
| - Be polite, conversational, and act as an active pairing partner. | ||
| - Use natural transitions and acknowledge the user's goals before diving into code. | ||
|
|
||
| ## 2. Skepticism & Teaching | ||
| - If the user asks for something that violates Terraform or Go best practices, politely push back and explain why. | ||
| - Suggest better alternatives, workflows, or commands that might suit their goals better. | ||
|
|
||
| ## 3. Clarity | ||
| - When providing code or diffs, clearly explain the "why" behind the implementation. | ||
| - Break down complex logic into easy-to-understand bullet points. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # AI Agent Rules | ||
|
|
||
| This directory contains strict coding standards, anti-patterns, and requirements based on file types. | ||
|
|
||
| AI agents MUST consult the corresponding instruction file in this directory (e.g., `go.instructions.md`, `terraform.instructions.md`) whenever asked to generate, edit, or review code. |
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # GitHub Copilot Review Instructions | ||
|
|
||
| When performing a code review or suggesting changes, adhere to the following guidelines to avoid "bikeshedding" and ensure feedback remains high-impact: | ||
|
|
||
| ## Focus on Critical and Highly Important Issues | ||
| - **Security:** Highlight potential vulnerabilities, exposed secrets, or unsafe data handling. | ||
| - **Bugs & Logic Errors:** Point out broken logic, unhandled edge cases, nil pointer dereferences, or potential race conditions. | ||
| - **Performance:** Identify significant bottlenecks, severe memory leaks, or highly inefficient resource usage. | ||
| - **Architecture:** Flag major architectural flaws or severe violations of core design principles that will drastically harm maintainability. | ||
|
|
||
| ## Avoid Bikeshedding (Trivial Suggestions) | ||
| - Do **not** suggest changes that minimally affect the functionality of the code. | ||
| - Ignore subjective styling, variable naming (unless dangerously misleading), and minor formatting adjustments. | ||
| - Do not recommend alternative language syntax or minor refactors if the current implementation is functional and readable. | ||
| - If a suggestion does not prevent a bug, fix a vulnerability, or drastically improve performance, omit it. | ||
|
|
||
| ## Review Format | ||
| - Provide actionable, concrete feedback for the critical issues identified. | ||
| - If the pull request has no critical or highly important issues, explicitly state that the code looks good and approve the review. | ||
| - Resist the urge to leave comments just for the sake of leaving comments. |
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # AI Agent Skills | ||
|
|
||
| This directory contains reusable tools or scripts that AI agents can recommend or utilize to execute tasks in this repository. | ||
|
|
||
| Examples include scripts for running acceptance tests (`run-acc-test.sh`), linting code, or automating repetitive tasks. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| #!/usr/bin/env bash | ||
| # | ||
| # Skill: run-acc-test.sh | ||
| # Description: Safely runs a specific Terraform provider acceptance test within the Nix environment. | ||
| # It explicitly keeps necessary AWS and Terraform environment variables so tests don't fail. | ||
| # Usage: ./run-acc-test.sh <TestName> | ||
|
|
||
| set -euo pipefail | ||
|
|
||
| if [ -z "${1:-}" ]; then | ||
| echo "Error: Must provide a test name." | ||
| echo "Usage: $0 <TestName>" | ||
| exit 1 | ||
| fi | ||
|
|
||
| TEST_NAME="$1" | ||
|
|
||
| echo "Executing acceptance test: ${TEST_NAME} inside Nix environment..." | ||
|
|
||
| nix develop --ignore-environment \ | ||
| --extra-experimental-features nix-command \ | ||
| --extra-experimental-features flakes \ | ||
| --keep HOME --keep SSH_AUTH_SOCK --keep GPG_SIGNING_KEY --keep NIX_SSL_CERT_FILE --keep NIX_ENV_LOADED --keep TERM \ | ||
| --keep AWS_ROLE --keep AWS_REGION --keep AWS_DEFAULT_REGION \ | ||
| --keep AWS_ACCESS_KEY_ID --keep AWS_SECRET_ACCESS_KEY --keep AWS_SESSION_TOKEN \ | ||
| --keep TF_VAR_aws_access_key_id --keep TF_VAR_aws_secret_access_key --keep TF_VAR_aws_session_token --keep TF_VAR_aws_region \ | ||
| --command bash -c "./run_tests.sh -t ${TEST_NAME}" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| #!/usr/bin/env bash | ||
| # | ||
| # Skill: run-in-nix.sh | ||
| # Description: Executes a given command inside the standardized Nix development environment. | ||
| # Usage: ./run-in-nix.sh "<command>" | ||
|
|
||
| set -euo pipefail | ||
|
|
||
| if [ $# -eq 0 ]; then | ||
| echo "Error: Command required." | ||
| echo "Usage: $0 \"<command>\"" | ||
| exit 1 | ||
| fi | ||
|
|
||
| COMMAND="$1" | ||
|
|
||
| echo "Running command in Nix environment: ${COMMAND}" | ||
|
|
||
| nix develop --ignore-environment \ | ||
| --extra-experimental-features nix-command \ | ||
| --extra-experimental-features flakes \ | ||
| --keep HOME --keep SSH_AUTH_SOCK --keep GPG_SIGNING_KEY --keep NIX_SSL_CERT_FILE --keep NIX_ENV_LOADED --keep TERM \ | ||
| --command bash -c "${COMMAND}" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # AI Agent Workflows | ||
|
|
||
| This directory contains defined processes for executing multi-step tasks. | ||
|
|
||
| These workflows provide step-by-step procedures for AI agents to follow when tackling complex tasks such as releasing the provider, running full test suites, or scaffolding new Terraform resources. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # GitHub Copilot Instructions | ||
|
|
||
| Before analyzing this repository, providing code suggestions, or reviewing pull requests, you MUST read the authoritative root prompt for all agents located in `AGENTS.md` at the root of this repository. | ||
|
|
||
| It contains essential coding standards, the `.agent` directory structure you need to use, and strict rules specific to your role in this project. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,17 +1,17 @@ | ||
| <!--- If there is no user issue related to this then you should remove the next line ---> | ||
| Addresses # | ||
|
|
||
| <!--- Add labels (eg. release/v14) for each release branch to target ---> | ||
| - Addresses # | ||
| <!--- Add labels (eg. release/v15) for each release branch to target ---> | ||
| <!--- Please don't manually add "internal" labels, those are for automation only ---> | ||
|
|
||
| ## Description | ||
|
|
||
| <!--- Describe your change and how it addresses the issue linked above or a problem with the product. ---> | ||
|
|
||
| ## Testing | ||
|
|
||
| ## Testing | ||
| <!--- Please describe how you verified this change or why testing isn't relevant. ---> | ||
|
|
||
|
|
||
| <!--- Does this change alter an interface that users of the provider will need to adjust to? ---> | ||
| <!--- Will there be any existing configurations broken by this change? If so, change the following line with an explanation. ---> | ||
| <!--- Will there be any existing configurations broken by this change? --> | ||
| <!--- If so, change the following line with an explanation. ---> | ||
| Not a breaking change. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.