Azure / Azure/azure-sdk-tools

Temp files should go to $TMP instead of TempTypeSpecFiles

Open
#9,248 0 comments 0 reactions 0 assignees View on GitHub
TypeSpec
Dominant language
C#
Stars
135
Forks
260
Avg merge
3d 1h
Merged PRs (30d)
143

Description

Why are we create directories named `TempTypeSpecFiles` in the project workspace? This may be in the workspace root or other directories depending on the CWD, it seems. Why aren't we putting them in `$TMP`? If these are incremental, a common solution is to hash the CWD path (case-sensitive for linux, mac; case-insensitive for Windows) as part of the directory name or under the directory.

Sure, we can `.gitignore` "TempTypeSpecFiles" but grep'ing for source or some LSPs that parse files just get more inundated with cruft. It also no doubt creates problem, if incremental, when switching branches (though, I suppose only including the CWD hash would do so, so maybe the branch name should be considered from `git branch --show-current`).

Contributor guide

Open the contributing guide

Research direction

Start by locating the code that creates directories named TempTypeSpecFiles and inspect how it chooses the directory from the current working directory. Confirm whether the files are incremental, then determine how temporary storage should avoid workspace cruft while preserving the required behavior across operating systems; done means these directories no longer appear in project workspaces.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.