TypeError reading metadata
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- typescript
- Domain
- cli
Research direction
Start with AppMapBuilder.normalize in node_modules/@appland/models/dist/index.cjs and the scanner entry point in node_modules/@appland/scanner/built/cli/scan.js, then reproduce the yarn scanner scan --watch command. Done means scanning no longer throws a TypeError when reading metadata; add or run a focused test if the repository provides one.
Written by the indexing model from the issue text.
Description
[stderr] /Users/kgilpin/.nvm/nvm-exec yarn run scanner scan --watch --appmap-dir .: /Users/kgilpin/source/appland/vscode-appland/node_modules/@appland/models/dist/index.cjs:3655
if (/^https?/.test(this.data.metadata?.git?.repository)) {
^
TypeError: Cannot read properties of undefined (reading 'metadata')
at AppMapBuilder.normalize (/Users/kgilpin/source/appland/vscode-appland/node_modules/@appland/models/dist/index.cjs:3655:34)
at /Users/kgilpin/source/appland/vscode-appland/node_modules/@appland/scanner/built/cli/scan.js:51:66
at Generator.next (<anonymous>)
at fulfilled (/Users/kgilpin/source/appland/vscode-appland/node_modules/@appland/scanner/built/cli/scan.js:5:58)
// Remove credentials from git repository url
if (/^https?/.test(this.data.metadata?.git?.repository)) {
const url = new URL(this.data.metadata.git.repository);
url.username = '';
url.password = '';
this.data.metadata.git.repository = url.toString();
}
- Dominant language
- TypeScript
- Stars
- 53
- Forks
- 18
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 8
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from getappmap/appmap-js
-
Difficulty 4/5 3-5 days Newbie friendliness 58/100
-
Difficulty 3/5 1-2 days Newbie friendliness 38/100
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
All issues in getappmap/appmap-js
Similar issues
-
comp/dashboard P3 type/bug
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
NousResearch/hermes-agent#117722 ·
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 76/100
-
code-quality refactoring
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
github/gh-aw-firewall#8816 ·
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
langchain-ai/deepagents#6450 ·