List every runtime global and align Performance, goccia:ast and CSV/TSV parseChunk behaviour
Maintainers usually reply within 2 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 52/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript
- Domain
- compilers, testing-qa
Research direction
Start by tracing runtimeGlobals and the Performance constructor, then inspect goccia:ast parse and the CSV/TSV parseChunk implementations. Done means all installed globals are listed, both Performance forms share the expected error, syntax errors include fileName, and CSV/TSV agree on resumable final rows with tests covering each case.
Written by the indexing model from the issue text.
Description
Summary
The built-in surface has a few inconsistencies. Found in the 0.14.0 docs audit.
Current behavior
-
runtimeGlobalsis incomplete.Goccia.runtimeGlobalslists only globals registered throughRegisterRuntimeGlobalName: EventTarget, Event, AbortController, AbortSignal, plus FFI, timers or test assertions when installed. Console, Performance, TextEncoder/TextDecoder, URL/URLSearchParams, fetch, Headers and Response never register. -
Performanceconstructor messages differ.new Performance()throwsPerformance is not a constructor, whilePerformance()throwsIllegal constructor. -
goccia:astdrops the file name.parse(src, { fileName })never puts the file name in the syntax-error message (Expected variable name (1:7));fileNameonly affects the JSX extension warning. -
CSV.parseChunkandTSV.parseChunkdisagree on an unterminated final row:- CSV consumes it:
parseChunk("a,b\n1,2\n3")returnsdone: truewith{a:"3", b:""}. - TSV leaves it unread:
done: false.
Neither is tested, and the CSV behaviour contradicts the resume model JSONL documents.
- CSV consumes it:
Expected behavior
runtimeGlobalslists every runtime global the host installed.Performancethrows the same "Illegal constructor"TypeErrorin both forms.goccia:astincludesfileNamein syntax-error messages.- CSV and TSV
parseChunkagree, most likely on TSV's resumable behaviour, with tests.
Scope notes
The docs are being corrected to describe the current behaviour until these are fixed.
- Dominant language
- Pascal
- Stars
- 20
- Forks
- 3
- Avg merge
- 3d 21h
- Merged PRs (30d)
- 51
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
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 frostney/GocciaScript
-
documentation internal
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
frostney/GocciaScript#1271 ·
Maintainers usually reply within 2 days
-
internal
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
frostney/GocciaScript#1187 · 1 comment ·
Maintainers usually reply within 2 days
-
bug internal
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
frostney/GocciaScript#1129 ·
Maintainers usually reply within 2 days
-
engine
Difficulty 5/5 Over a week Newbie friendliness 35/100
frostney/GocciaScript#1279 · 2 comments ·
Maintainers usually reply within 2 days
-
runtime
Difficulty 4/5 3-5 days Newbie friendliness 52/100
frostney/GocciaScript#1278 · 1 comment ·
Maintainers usually reply within 2 days
All issues in frostney/GocciaScript
Similar issues
-
backend:DirectX
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
llvm/llvm-project#227530 ·
Maintainers usually reply within 1 day
-
`enzymexla.linalg.lu` lowering fails for a tall matrix: the permutation is built with the pivot typeOpen
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
EnzymeAD/Enzyme-JAX#3286 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
objectionary/phino#1600 ·
Maintainers usually reply within 1 day
-
compiler enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
tenstorrent/tt-lang#1141 ·
Maintainers usually reply within 5 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day