Skip to content

cleanup: remove accidentally-nested amplifier-foundation/ directory from PR #160#174

Merged
bkrabach merged 1 commit into
mainfrom
cleanup/remove-nested-directory
Apr 20, 2026
Merged

cleanup: remove accidentally-nested amplifier-foundation/ directory from PR #160#174
bkrabach merged 1 commit into
mainfrom
cleanup/remove-nested-directory

Conversation

@bkrabach

Copy link
Copy Markdown
Collaborator

Problem

PR #160 (merge commit b10b8e73) accidentally committed a full nested copy of the repo at amplifier-foundation/ inside the repo root. Two levels of self-nesting, 610 stale files. Dependabot has been generating PRs against files inside the nested paths since.

Verification (nothing legit at risk)

  • pyproject.toml is identical between root and nested copies
  • uv.lock is identical between root and nested copies
  • Nested amplifier_foundation/configurator/__init__.py snapshots are the older monolithic (643 and 673 line) dev versions; the refactored package at the root (231-line facade + _state_manager + _inspector + _provenance_utils) is current and unaffected

Changes

  • git rm -rf amplifier-foundation/ — removes the entire nested tree
  • Adds /amplifier-foundation/ to .gitignore to prevent recurrence

Follow-up

Closing these stale Dependabot PRs (targeting nested paths that will not exist after merge): #161, #162, #163, #164, #165, #166, #167, #168, #169, #170.

Keeping the 5 Dependabot PRs that target real root paths (#139, #141, #142, #156, #157 — for /modules/hooks-process-guard and root).

The SessionConfigurator feature itself from PR #160 is unaffected — this is pure cleanup of collateral pollution.

PR #160 (merge commit b10b8e7) accidentally committed a full copy of
the repo at amplifier-foundation/ inside the repo root — two levels of
self-nesting, 610 stale files.

Verified no legit content at risk:
- pyproject.toml IDENTICAL between root and nested copies
- uv.lock IDENTICAL between root and nested copies
- Nested configurator snapshots are older dev states (monolithic), not
  newer; the refactored package at amplifier_foundation/ (root) is current

Also adds /amplifier-foundation/ to .gitignore to prevent recurrence from
accidentally cloning the repo into its own working tree.

10 stale Dependabot PRs (#161, #162, #163, #164, #165, #166, #167, #168,
#169, #170 — all targeting /amplifier-foundation paths) will be closed as
no longer applicable.
@bkrabach bkrabach merged commit e06dce6 into main Apr 20, 2026
4 checks passed
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.

1 participant