Skip to content

chore: remove repository scaffolding artifacts and relocate .NET local tools manifest - #2

Merged
Devzinh merged 2 commits into
mainfrom
copilot/apply-repository-cleanup
Jul 24, 2026
Merged

chore: remove repository scaffolding artifacts and relocate .NET local tools manifest#2
Devzinh merged 2 commits into
mainfrom
copilot/apply-repository-cleanup

Conversation

Copilot AI commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

This PR applies a scoped repository cleanup: removes empty scaffolding/sample content and accidental scan/debug artifacts, and relocates the local .NET tools manifest to the canonical path used by dotnet tool restore.

  • Cleanup: empty scaffolding and placeholders

    • Removed samples/ placeholder tree (datasets/, profiles/, reports with only .gitkeep).
    • Removed placeholder test projects/directories:
      • tests/GhostTrace.Tests.Integration/
      • tests/GhostTrace.Tests.ForensicSafety/
    • Removed placeholder unit-test artifacts in tests/GhostTrace.Tests.Unit/:
      • UnitTest1.cs
      • CLI/.gitkeep, Core/.gitkeep, Modules/.gitkeep
    • Removed empty CI scaffold marker: tools/ci/.gitkeep.
  • Cleanup: accidental/unneeded tracked files

    • Removed root scan artifact: correlation.json.
    • Removed unused README asset: assets/readme/hero.gif.
  • Tooling manifest path fix

    • Moved local tools manifest:
      • dotnet-tools.json.config/dotnet-tools.json
    • This aligns with .NET local tool discovery behavior.
  • Ignore rules

    • Confirmed required scan-output rules are present in .gitignore:
      • correlation.json
      • *-report.json
      • *-report.txt
      • cleanup-*.log
mkdir -p .config
git mv dotnet-tools.json .config/dotnet-tools.json
  • Validation required by scope
    • dotnet tool restore succeeds with the manifest in .config/.
    • dotnet build GhostTrace.sln still fails on Linux at the existing Windows-only WiX installer project (GhostTrace.Installer.wixproj), unchanged by this cleanup.

Copilot AI changed the title [WIP] Apply repository cleanup for GhostTrace chore: remove repository scaffolding artifacts and relocate .NET local tools manifest Jul 24, 2026
Copilot AI requested a review from Devzinh July 24, 2026 21:41
@Devzinh
Devzinh marked this pull request as ready for review July 24, 2026 21:46
Copilot AI review requested due to automatic review settings July 24, 2026 21:46
@Devzinh
Devzinh merged commit c37941f into main Jul 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 performs repository housekeeping by removing empty scaffolding/placeholder artifacts and accidental scan output, while relocating the .NET local tools manifest to .config/dotnet-tools.json so dotnet tool restore discovers it via the canonical path.

Changes:

  • Removed placeholder sample folders and placeholder/empty test project artifacts (including empty xUnit stubs and .gitkeep markers).
  • Removed an accidentally committed scan artifact (correlation.json) and an empty CI scaffold marker.
  • Added the local .NET tools manifest under .config/dotnet-tools.json to align with dotnet tool restore discovery rules.

Reviewed changes

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

Show a summary per file
File Description
tools/ci/.gitkeep Removes empty CI scaffold marker.
tests/GhostTrace.Tests.Unit/UnitTest1.cs Deletes placeholder unit test stub.
tests/GhostTrace.Tests.Unit/Modules/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.Unit/Core/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.Unit/CLI/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.Integration/UnitTest1.cs Deletes placeholder integration test stub.
tests/GhostTrace.Tests.Integration/Reporting/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.Integration/Pipeline/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.Integration/GhostTrace.Tests.Integration.csproj Removes placeholder integration test project definition.
tests/GhostTrace.Tests.Integration/Fixtures/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.ForensicSafety/UnitTest1.cs Deletes placeholder forensic-safety test stub.
tests/GhostTrace.Tests.ForensicSafety/RegistryAccess/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.ForensicSafety/ReadOnlyGuards/.gitkeep Removes empty placeholder directory marker.
tests/GhostTrace.Tests.ForensicSafety/GhostTrace.Tests.ForensicSafety.csproj Removes placeholder forensic-safety test project definition.
tests/GhostTrace.Tests.ForensicSafety/FilesystemAccess/.gitkeep Removes empty placeholder directory marker.
samples/reports/.gitkeep Removes empty placeholder directory marker.
samples/profiles/.gitkeep Removes empty placeholder directory marker.
samples/datasets/.gitkeep Removes empty placeholder directory marker.
correlation.json Removes accidentally tracked scan artifact from repo root.
.config/dotnet-tools.json Adds local tool manifest in the canonical .config/ location for dotnet tool restore.

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

@Devzinh
Devzinh deleted the copilot/apply-repository-cleanup branch July 24, 2026 22:15
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.

3 participants