issues Search Results · language:Dune language:JavaScript language:JavaScript language:Java language:JavaScript is:public
Filter by
33.3M results
Reorganize into types/{schema,content,media,options}.ts with a barrel (pure move — only types/index.js is imported
anywhere).
Site has an open [key: string]: unknown that defeats typo-checking on its ...
P2
refactor
page-editor.ts (938 lines): one closure over shared mutable locals → extract PagesTableController, BlockListController,
PageFormController; replace the stringly-typed 3-part/4-part error-key bridge with ...
P2
refactor
scripts/coverage.mjs computes a % for the README badge but never fails on regression, and it does not run in CI at all.
A coverage drop can ship to npm unnoticed.
Fix: add a --check threshold mode; run ...
P2
tooling
Verified absent: no ESLint/Prettier/Biome config or dependency. There is even an orphaned // eslint-disable-next-line in
scripts/coverage.mjs. Only .editorconfig exists, enforced by nothing.
Fix: adopt ...
P2
tooling
Build Scans:
- elasticsearch-intake #42627 / part3
Reproduction Line:
./gradlew :x-pack:plugin:esql:test --tests org.elasticsearch.xpack.esql.expression.predicate.operator.arithmetic.DivTests.testCrankyEvaluateBlockWithoutNulls ...
:Analytics/ES|QL
low-risk
Team:Analytics
>test-failure
Catalog parity: routes/admin/i18n/types.ts types Catalog as a bare Record string,string , so es satisfies Catalog only
checks value types, not key completeness — parity is guarded only by a runtime test. ...
P1
refactor
Where: escapeHtml in common.ts (text-safe) vs escapePickerHtml in block-form.ts (attribute-safe) vs a third
escapeHtml/escapeAttr reimplemented in media.ts. The block-form doc comment already warns callers ...
P1
refactor
security
Where: routes/admin/client/block-form.ts owns a singleton picker dialog, field renderers, DOM patchers, event wiring,
array-limit logic, and MIME computation — plus a 650-line style string built inside ...
P1
refactor
Where: handleDeleteLanguage / handlePostLanguages / handlePutLanguage in api/handlers.ts perform no internal role check
— unlike handlePostUsers/handleExport/handleImport, which call requireOwner internally ...
P1
refactor
security
Where: routes/api/catchall.ts — ~40 if (seg[0] === x seg.length === N) branches; requireOwner inlined ad hoc per route.
Risk: adding a route means remembering to inline the owner-check in the right verb ...
P1
refactor

Learn how you can use GitHub Issues to plan and track your work.
Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub IssuesProTip! Restrict your search to the title by using the in:title qualifier.