getappmap / getappmap/appmap-js

Context selection excludes the content of Pinned files

Open
#1,984 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

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

Description

When selecting context, the contents of "pinned" files should not be included:

Screen Shot 2024-09-11 at 2 42 45 PM

--

In the navie package, set context options indicating which files are already "pinned". Utilize the existing "exclude" list option for this.

To support this option, the "codeSelection" parameter should be enhanced to support named files.

In the cli package, skip "pinned" files when searching for context.

Ignore the UI (components) for this design.

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.

Research direction

Start in the navie package at context options and the codeSelection parameter, then trace the cli package's context search. Done means named pinned files are passed through the existing exclude list and their contents are omitted from context, without changing UI components.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
cli, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.