Skip to content

lookup for generated files: support file names used by automake #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
theodevelop merged 5 commits into from
Apr 17, 2026
Merged

lookup for generated files: support file names used by automake #45

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1b85395
lookup for generated files: support file names used by automake
GitMensch Apr 16, 2026
42f41a4
allow CI to be run manually on all branches
GitMensch Apr 16, 2026
48a57d2
Update lineDirectiveNavigation.ts
GitMensch Apr 16, 2026
9acf2e4
Update lineDirectiveNavigation.ts
GitMensch Apr 16, 2026
daa7480
Merge branch 'dev' into patch-1
theodevelop Apr 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
branches: ["main", "dev"]
pull_request:
branches: ["main", "dev"]
# manual run in actions tab - for all branches
workflow_dispatch:

jobs:
Expand Down
25 changes: 22 additions & 3 deletions client/src/lineDirectiveNavigation.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,23 @@ const BISON_CANDIDATES = (base: string, dir: string): string[] => [
path.join(dir, base + '.tab.c'),
path.join(dir, base + '.tab.cpp'),
path.join(dir, base + '.tab.cc'),
path.join(dir, base + '.c'),
path.join(dir, base + '.cc'),
path.join(dir, base + '.c++'),
path.join(dir, base + '.cxx'),
path.join(dir, base + '.cpp'),
];

const FLEX_CANDIDATES = (base: string, dir: string): string[] => [
path.join(dir, 'lex.yy.c'),
path.join(dir, 'lex.yy.cc'),
path.join(dir, base + '.yy.c'),
path.join(dir, base + '.yy.cpp'),
path.join(dir, base + '.c'),
path.join(dir, base + '.cc'),
path.join(dir, base + '.c++'),
path.join(dir, base + '.cxx'),
path.join(dir, base + '.cpp'),
];

/** Scan CMakeLists.txt up the directory tree and return a build directory hint. */
Expand Down Expand Up @@ -132,7 +142,7 @@ async function findGeneratedFile(sourceFilePath: string): Promise<string | null>
const sourceDir = path.dirname(sourceFilePath);
const base = path.basename(sourceFilePath, path.extname(sourceFilePath));
const ext = path.extname(sourceFilePath).toLowerCase();
const isBison = ['.y', '.yy', '.ypp', '.bison'].includes(ext);
const isBison = ['.y', '.yy', '.y++', '.ypp', '.yxx', '.bison'].includes(ext);
const candidates = isBison ? BISON_CANDIDATES : FLEX_CANDIDATES;

const dirsToTry: string[] = [];
Expand All @@ -150,8 +160,17 @@ async function findGeneratedFile(sourceFilePath: string): Promise<string | null>
}

// Workspace-wide search as last resort — let user pick if ambiguous
const pattern = isBison ? `**/${base}.tab.{c,cpp,cc}` : `**/lex.yy.{c,cc}`;
const found = await workspace.findFiles(pattern, '**/node_modules/**', 10);
let pattern = isBison ? `**/${base}.tab.{c,cpp,cc}` : `**/lex.yy.{c,cc}`;
let found = await workspace.findFiles(pattern, '**/node_modules/**', 10);

// in case of no results, check for ylwrap names (no tab/lex)
// we only do that after the initial run to not force a picker if
// the tool's default names are available somewhere in the workspace
if (found.length === 0) {
pattern = `**/${base}.{c,cc,c++,cxx,cpp}`;
found = await workspace.findFiles(pattern, '**/node_modules/**', 10);
}

if (found.length === 0) return null;
if (found.length === 1) return found[0].fsPath;

Expand Down
Loading