Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 8, 2026

Dry-run audit of the repo surfaced three categories of drift between actual file state and what scripts/configs reference.

bootstrap.sh

  • Removed reference to non-existent QUICKSTART.md in show_next_steps(), replaced with README.md Quick Start section

cleanup.sh

  • Added 18 missing entries to EXPECTED_FILES: all symlink READMEs (14), src/scripts/performance/* (3), and .jules/bolt.md — these were added in prior PRs but the expected file list was never updated

terraform/main.tf

  • Removed duplicate required_version + required_providers block already defined in versions.tf. Kept the empty terraform {} block for the commented-out backend config.
# Before: main.tf duplicated the entire terraform {} block from versions.tf
# After: main.tf only retains the backend placeholder
terraform {
  # Uncomment and configure backend for remote state management
  # backend "s3" { ... }
}
Original prompt

perform a dry run of this repo and check for missing or redundent lines and args

Custom agent used: documentation-builder
Specialized agent for building README.md and AGENTS.md files from template versions


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

…, remove redundant terraform block

Co-authored-by: swarm-protocol <169869917+swarm-protocol@users.noreply.github.com>
Copilot AI changed the title [WIP] Perform dry run to check for missing or redundant lines and args fix: dry run audit — stale references, missing expected files, redundant terraform block Feb 8, 2026
Copilot AI requested a review from swarm-protocol February 8, 2026 12:09
@swarm-protocol swarm-protocol marked this pull request as ready for review February 8, 2026 12:10
Copilot AI review requested due to automatic review settings February 8, 2026 12:10
@swarm-protocol swarm-protocol merged commit 37ff26d into main Feb 8, 2026
4 checks passed
@swarm-protocol swarm-protocol deleted the copilot/check-dry-run-repo branch February 8, 2026 12:10
Copy link
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

Aligns repo scripts/config with the current file layout by removing stale documentation references, updating the cleanup allowlist, and de-duplicating Terraform version/provider constraints.

Changes:

  • bootstrap.sh: Replace stale QUICKSTART.md reference with a pointer to the README “Quick Start” section.
  • cleanup.sh: Add missing entries to the (commented) EXPECTED_FILES allowlist so dry-run audits don’t flag existing files as “leftovers”.
  • terraform/main.tf: Remove duplicated required_version/required_providers (now centralized in versions.tf) while keeping the backend placeholder block.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
bootstrap.sh Removes reference to a non-existent doc and points users to README’s Quick Start section.
cleanup.sh Updates the expected file allowlist to reflect files that already exist in the repo.
terraform/main.tf Eliminates redundant Terraform constraints duplicated in versions.tf, retaining backend placeholder.

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

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