getappmap / getappmap/appmap-js

Delete old AppMap binaries

Open
#2,178 1 comment 0 reactions 1 assignee View on GitHub

@zermelo-wisen is already working on this.

Since Aug 26, 2024.

bug
Dominant language
TypeScript
Stars
53
Forks
18
Avg merge
3d 18h
Merged PRs (30d)
8

Description

Problem

The AppMap plugin currently does not clean up older versions of AppMap binaries located in the $HOME/.appmap (Linux/macOS) or %HOME%/.appmap (Windows) directory. This can lead to unnecessary storage usage as multiple old versions accumulate over time. The goal is to remove all older versions of the binaries and ensure that only the most recent version is retained moving forward.

Analysis

To resolve this issue, we need to implement a cleanup function that scans the AppMap binaries directory and deletes all but the most recent binary. The most recent binary can be determined by the semver version number in the file name.

Furthermore, we should integrate this cleanup function to be automatically executed whenever a new AppMap binary is downloaded or installed. This ensures that the directory is consistently maintained with only the latest version, preventing storage bloat.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.