Skip to content

Commit f44fa39

Browse files
chore(report): update TODO report and changelog [skip ci]
1 parent 8a7f146 commit f44fa39

File tree

1 file changed

+10
-249
lines changed

1 file changed

+10
-249
lines changed

TODO_REPORT.md

Lines changed: 10 additions & 249 deletions
Original file line numberDiff line numberDiff line change
@@ -1,267 +1,28 @@
11
# 📌 TODO Report
22

3-
Total unique TODOs: **236**
3+
Total unique TODOs: **12**
44

55
## TODO
66

7-
- `inline.js:49` — Refactor login logic @alice #auth type=refactor _( priority=high, due=2025-06-01 )_
7+
- `inline.ts:14` — Refactor login logic @alice #auth type=refactor _( priority=high, due=2025-06-01 )_
88
- `inline.ts:1` — Refatorar este método _( priority=high, due=2025-06-01 )_
9-
- `inline.js:74` — improve retry logic for API errors _( priority=high )_
10-
- `inline.js:4` — Refactor this module
11-
- `inline.js:2227` — type `options.auth` based on `options.authStrategy`.
12-
- `inline.js:3994` — support preAuth = false where it hooks on 401
13-
- `inline.js:5413` — create separate package.
14-
- `inline.js:8105` — need to fix on node itself
15-
- `inline.js:9660` — Use BigInt if supported?
16-
- `inline.js:10363` — Test controller argument
17-
- `inline.js:10683` — Ideally in this branch detaching should happen only if the buffer is not consumed fully.
18-
- `inline.js:11472` — Fix alphabetical order.
19-
- `inline.js:17543` — Does this need queueMicrotask?
20-
- `inline.js:17721` — Avoid finished. It registers an unnecessary amount of listeners.
21-
- `inline.js:18082` — Implement.
22-
- `inline.js:18096` — Is this the best way to force a lock?
23-
- `inline.js:19154` — the spec is wrong, this is needed to pass WPTs
24-
- `inline.js:19983` — for H2 we need to gracefully flush the remaining enqueued
25-
- `inline.js:20004` — (fix): Should we error here with ClientDestroyedError?
26-
- `inline.js:21257` — Should we call onConnect immediately or on stream ready event?
27-
- `inline.js:21300` — unref only if current streams count is 0
28-
- `inline.js:21427` — Support aborted
29-
- `inline.js:21431` — Support timeout
30-
- `inline.js:21435` — Suppor push
31-
- `inline.js:21439` — Support trailers
32-
- `inline.js:22774` — session re-use does not wait for the first
33-
- `inline.js:22868` — Add support for h2c
34-
- `inline.js:22877` — (fix): Can a session become invalid once established? Don't think so?
35-
- `inline.js:23638` — (fix): This might be a bad idea?
36-
- `inline.js:23670` — adjust to support H2
37-
- `inline.js:23677` — Migrate header parsing here, to make Requests
38-
- `inline.js:26171` — argument idl type check
39-
- `inline.js:26190`
40-
- `inline.js:27430` — (fix): Find and fix root cause for leaked listener.
41-
- `inline.js:27649` — given global’s relevant settings object’s cross-origin isolated
42-
- `inline.js:27792` — What if request.client is null?
43-
- `inline.js:27900` — should fetching request be blocked as mixed content?
44-
- `inline.js:27901` — should request be blocked by Content Security Policy?
45-
- `inline.js:28613` — cache
46-
- `inline.js:28756`https://github.com/whatwg/fetch/issues/1285#issuecomment-896560129
47-
- `inline.js:28771` — credentials
48-
- `inline.js:28777` — proxy-authentication
49-
- `inline.js:28876` — Invoke some kind of callback?
50-
- `inline.js:28903` — (spec): The spec doesn't specify this but we need to cancel
51-
- `inline.js:29320` — (fix): Do we need connection here?
52-
- `inline.js:29848` — could this be simplified with AbortSignal.any
53-
- `inline.js:30552` — base-URL?
54-
- `inline.js:32058` — add comment explaining why this error occurs.
55-
- `inline.js:34140` — (fix): Provide some way for the user to cache the file to e.g. /tmp
56-
- `inline.js:34142` — (fix): Do we need 100-expect support to provide a way to do this properly?
57-
- `inline.js:34157` — (fix): We can't access ReadableStream internal state
58-
- `inline.js:34167` — Should we allow re-using iterable if !this.opts.idempotent
59-
- `inline.js:34852` — should we allow it with HTTP?
60-
- `inline.js:34913` — use RFC
61-
- `inline.js:36930` — enable once permessage-deflate is supported
62-
- `inline.js:37912` — optimize this
63-
- `inline.js:39075` — query stuff
64-
- `inline.js:45266` — handle blob, arraybuffer, other content types, etc.
65-
- `inline.js:45840` — add support for Cloudflare workers, etc.
66-
- `inline.js:47634` — clean this logic up
67-
- `inline.js:47738` — support audio here
68-
- `inline.js:48973` — these types are incompatible
69-
- `inline.js:52810` — Is this where the error should be thrown?
70-
- `inline.js:53197` — make nested formats configurable
71-
- `inline.js:53494` — (jimmywarting): in the feature use conditional loading with top level await (requires 14.x)
72-
- `inline.js:5` — Refactor this logic to improve performance
73-
- `inline.js:40` — Eventually support sourceRoot, which has to be removed because the sources are already
74-
- `inline.js:138` — We should eventually support async loading of sourcemap files.
75-
- `inline.js:111` — Binary search (check overhead)
76-
- `inline.js:138` — un-copy these from resolve-uri; see if they can be exported from that lib
77-
- `inline.js:392` — Remove now that windows path support was added to resolve-uri and thus trace-mapping?
78-
- `inline.js:629` — is there a better way to reliably get an instance of NodeError?
79-
- `inline.js:452` — Use performance.now() once it's mocked
80-
- `inline.js:824` — do we need to define the generics twice?
81-
- `inline.js:129` — type `octokit.auth` based on passed options.authStrategy
82-
- `inline.ts:2` — remove once this package no longer supports TS 5.1, and replace with a
83-
- `inline.ts:2` — these methods are not used within @types/node, and should be removed at the next
84-
- `inline.ts:6` — remove once this package no longer supports TS 5.5, and replace NodeJS.BuiltinIteratorReturn with BuiltinIteratorReturn.
85-
- `inline.ts:97` — Add `EventEmitter` close
86-
- `inline.ts:439` — In next major @types/node version, change default TReturn to undefined.
87-
- `inline.ts:734` — consider removing in a future major version update
88-
- `inline.ts:46` — remove empty ConnectOpts placeholder at next major @types/node version.
89-
- `inline.ts:2227` — change the export to a wrapper function once node@0db38f0 is merged (breaking change)
90-
- `inline.ts:69` — remove in a future major version bump
91-
- `inline.js:2243` — investigate whether there's ever a case where we hit this logic with 1:many sources.
92-
- `inline.js:2423` — could we move the resolving logic for 1:1 source maps to the final
93-
- `inline.js:102` — is an absolute `cwd` supposed to be resolved against `root`?
94-
- `inline.ts:54` — Remove this for the next major release, refactor the whole definition to:
95-
- `inline.js:75` — Remove this for the next major release
96-
- `inline.js:122` — Looks like there is bug in Node fs.createReadStream
97-
- `inline.js:347` — instead of injecting the start/end at this point, just return
98-
- `inline.js:3` — increment this version if there are schema changes
99-
- `inline.js:3` — switch to class private field when targetting node.js 12
100-
- `inline.js:365` — check & read HasteFS instead of reading the filesystem:
101-
- `inline.js:28` — remove re-export in Jest 30
102-
- `inline.js:32` — in Jest 30 remove `SpyInstance` in favour of `Spied`
103-
- `inline.js:487` — build?
104-
- `inline.js:246` — Add appropriate type check
105-
- `inline.js:296` — At some point it would make sense to make use of
106-
- `inline.js:1621` — rework to inline fn with no type cast?
107-
- `inline.js:3002` — Add tag format check.
108-
- `inline.js:131` — is this block necessary?...
109-
- `inline.js:830` — last chunk in the array may not be the last chunk, if it's moved...
110-
- `inline.js:1029` — deprecate this? not really very useful
111-
- `inline.js:1310` — there's a bunch of this sort of thing, needs cleaning up
112-
- `inline.js:54` — use media-typer
113-
- `inline.js:78` — should this even be in this module?
114-
- `inline.js:91` — use content-type or other module
115-
- `inline.js:491` — It would probably be faster to determine this
116-
- `inline.js:65` — handle errors
117-
- `inline.ts:425` — These should be described using static getter/setter pairs:
118-
- `inline.ts:193` — a type could likely be implemented that can infer the return type
119-
- `inline.ts:30` — remove this in the next major version, refactor the whole definition to:
120-
- `inline.js:8` — remove this in the next major version
121-
- `inline.js:33` — test perf of fs/promises realpath vs realpathCB,
122-
- `inline.js:17440` — we are pretending it was not found while it should behave like "undefined"
123-
- `inline.js:6874` — emit errors properly. Example: EMFILE on Macos.
124-
- `inline.js:7508` — real check
125-
- `inline.js:7996` — Strange thing: "should not choke on an ignored watch path" will be failed without 2 ready calls -__-
126-
- `inline.js:8145` — this is not equal to path-normalize module - investigate why
127-
- `inline.js:9114` — throws or warn? Currently just ignore, uses new event
128-
- `inline.js:36` — fallback to string parsing?
129-
- `inline.js:454` — this comes from BootstrapState
130-
- `inline.js:304` — compute baseUrl
131-
- `inline.js:103` — file://./foo sets `hostname` to `'.'`. Perhaps we should special-case this.
132-
- `inline.js:227` — switch to getCanonicalFileName we already create later in scope
133-
- `inline.js:390` — ordering of this with getScriptVersion? Should they sync up?
134-
- `inline.js:126` — Figure out how to handle completion here.
135-
- `inline.js:156` — should evalCode API get the same error-handling benefits?
136-
- `inline.js:203` — assert that `service` is set; remove all `service!` non-null assertions
137-
- `inline.js:101` — consider using `ts.loadWithTypeDirectiveCache`
138-
- `inline.js:107` — cache the results of this; slightly faster
139-
- `inline.js:135` — technically breaks if projectOption is path to a file, not a directory,
140-
- `inline.js:20` — Use this set when resolving pkg#exports conditions in loader.js.
141-
- `inline.js:179` — Add the requireStack as well.
142-
- `inline.js:92` — receive cached fs implementations here
143-
- `inline.js:50` — handle errors somehow
144-
- `inline.js:10369` — don't use slice
145-
- `inline.js:14563` — These aren't valid TypeNodes, but we treat them as such because of `isPartOfTypeNode`, which returns `true` for things that aren't `TypeNode`s.
146-
- `inline.js:16003` — Should prefix `++` and `--` be moved to the `Update` precedence?
147-
- `inline.js:27152`GH#18217
148-
- `inline.js:30562` — JSDoc parameters don't have names (except `this`/`new`), should we manufacture an empty identifier?
149-
- `inline.js:54234`https://github.com/microsoft/TypeScript/pull/32372#discussion_r328386357
150-
- `inline.js:104221` — Does this need to be parented?
151-
- `inline.js:144642` — What should be the containerName when the container has a computed name?
152-
- `inline.js:156361` — Maybe we should handle this? See fourslash test `refactorConvertToEs6Module_export_object_shorthand.ts`.
153-
- `inline.js:164589` — Handle auto quote preference.
154-
- `inline.js:170969`GH#23879
155-
- `inline.js:174930` — should have its own kind?
156-
- `inline.js:192900` — correctly type the handlers
157-
- `inline.js:24431` — replace this whole reduce with a concat
158-
- `inline.js:24984` — is this correct?
159-
- `inline.js:26050` — Use `??=` when targeting Node.js 16.
160-
- `inline.js:44065` — Pass actual filename here, which can also be passed to esbuild's
161-
- `inline.js:2214` — Remove in next major version?
162-
- `inline.js:1874` — (special)
163-
- `inline.js:3691` — create home() in prompt types (e.g. TextPrompt)
164-
- `inline.js:3693` — create end() in prompt types (e.g. TextPrompt)
9+
- `inline.ts:40` — improve retry logic for API errors _( priority=high )_
10+
- `inline.ts:2` — Refactor this logic to improve performance
11+
- `inline.ts:1` — Refactor this module
16512

166-
## hack
167-
168-
- `inline.js:1470` — would
169-
- `inline.js:6966` — makes
170-
- `inline.js:7655` — for old IIS and Apache servers
171-
- `inline.js:16508` — y. This is necessary to avoid http-parser leaks.
172-
- `inline.js:42939` — because streams2 _always_ doesn't emit 'end' until nextTick, so let
173-
- `inline.js:54647` — makes specifying custom Host header possible.
174-
- `inline.js:194` — to prevent unexpected line breaks in the generated code
175-
- `inline.js:3220` — to not allow integers end with `_`
176-
- `inline.js:3296`
177-
- `inline.js:23563` — stolen from "mitt": ">>> 0" does not change numbers >= 0, but -1
178-
- `inline.js:66` — ; remove when we drop node12 support
179-
- `inline.js:74` — y workarounds, but it's not worth the complexity and flakiness.
180-
- `inline.js:17671` — and set the `readable` member to false.
181-
- `inline.js:32581` — is needed because the `__proto__` property is still inherited in
182-
- `inline.js:40240` — to avoid expensive fs checks for React apps.
183-
- `inline.js:41277` — to allow 'as' & 'query' exist at the same time
184-
185-
## bug
13+
## Todo
18614

187-
- `inline.js:8103` — fix here.
188-
- `inline.js:27691` — ?
189-
- `inline.js:28` — .js.map
190-
- `inline.js:19` — ging will be slightly less informative
191-
- `inline.js:760` — scheduling
192-
- `inline.js:412` — zilla.mozilla.org/show_bug.cgi?id=745678
193-
- `inline.js:6104` — tracker][ghbt] to report issues.
194-
- `inline.js:73` — in Bash, but since the goal of
195-
- `inline.js:134` — ? http://stackoverflow.com/a/398120/376773
196-
- `inline.js:228` — isn't set in LS, and we're in Electron, try to load $DEBUG
197-
- `inline.js:141` — instances
198-
- `inline.js:4087` — ger-statement
199-
- `inline.js:31` — mode. A backtrace uses ~1000 bytes of heap space and
200-
- `inline.js:209` — in V8 v6.X with --harmony enabled.
201-
- `inline.js:324` — ger to easily see which
202-
- `inline.js:137` — / --inspect flags while preserving others (like --harmony).
203-
- `inline.js:369` — in Jest itself.
204-
- `inline.js:104` — ging.
205-
- `inline.js:206` — ging and testing
206-
- `inline.js:32` — s that would otherwise result in us iterating indefinitely
207-
- `inline.js:11222` — ging
208-
- `inline.js:326` — zil.la/1090768.
209-
- `inline.js:331` — zil.la/1188982.
210-
- `inline.js:341` — zil.la/889492.
211-
- `inline.js:773` — zilla.mozilla.org/show_bug.cgi?id=885597.
212-
- `inline.js:539` — on old versions.
213-
- `inline.js:284` — .assert(array.length !== 0);
214-
- `inline.js:1210` — .ts
215-
- `inline.js:1647` — 's new customDescriptionGenerator in launch.json
216-
- `inline.js:24536` — s.chromium.org/p/v8/issues/detail?id=9560)
217-
- `inline.js:188313` — .js
218-
- `inline.js:189` — ger setup because by the time we run, node has already
219-
- `inline.js:18` — mode first before requiring the CLI.
220-
- `inline.js:30` — ging multiple flags with comma-separated list
221-
- `inline.js:19225` — gy property enumeration order in older V8 versions.
222-
- `inline.js:19227` — s.chromium.org/p/v8/issues/detail?id=4118
223-
- `inline.js:19234` — s.chromium.org/p/v8/issues/detail?id=3056
224-
- `inline.js:44211` — via `window` and `location` global
225-
- `inline.ts:4` — that makes <reference types="vite/types/importMeta" />
15+
- `inline.ts:1` — sFromContent.ts
16+
- `inline.ts:1` — sWithStructuredTags.ts
17+
- `inline.ts:1` — sWithStructuredTagsFromDir.ts
18+
- `inline.ts:1` — .ts
22619

22720
## FIXME
22821

229-
- `inline.js:9881` — Is this a bug in the specification, or in the test?
230-
- `inline.js:21926` — remove workaround when the Node bug is fixed
231-
- `inline.js:17669` — Should probably have an option in net.Socket to create a
232-
- `inline.js:17677` — Hack to have stream not keep the event loop alive.
23322
- `inline.ts:3` — Corrigir possível vazamento de memória
23423
- `inline.py:1` — Handle edge case
23524

236-
## Hack
237-
238-
- `inline.js:16513` — y.
239-
- `inline.js:590` — to work around lack of negative lookbehind in JS
240-
- `inline.js:141` — for first rule in CSS
241-
- `inline.js:3` — to avoid Module.runMain on node 18.6.0
242-
- `inline.js:17647` — to have stream not keep the event loop alive.
243-
244-
## Todo
245-
246-
- `inline.js:41664` — sFromContent.ts
247-
- `inline.js:41797` — sWithStructuredTagsFromDir.ts
248-
- `inline.js:4` — .ts
249-
- `inline.js:1` — (Babel 8): remove this file as Babel 8 drop support of core-js 2
250-
- `inline.js:1` — (Babel 8): remove this file now that it is included in babel-plugin-polyfill-corejs3
251-
- `inline.ts:1` — sWithStructuredTags.ts
252-
253-
## todo
254-
255-
- `inline.ts:231` — real token type
256-
- `inline.js:395` — is:
257-
25825
## BUG
25926

26027
- `inline.ts:6` — Corrigir lógica de ordenação _( due=2025-01-01 )_
261-
- `inline.js:89` — lDLENBU2xDOzs7QUFDQSxPQUFLLElBQUlDLENBQUMsR0FBRyxDQUFiLEVBQWdCQSxDQUFDLEdBQUdKLGdCQUFnQixDQUFDRSxNQUFyQyxFQUE2Q0UsQ0FBQyxFQUE5QyxFQUFrRDtBQUNoRCxRQUFJQyxJQUFJLEdBQUdMLGdCQUFnQixDQUFDSSxDQUFELENBQTNCOztBQUVBLFFBQUlBLENBQUMsR0FBRyxDQUFKLElBQVMsQ0FBQyxLQUFLRSxPQUFMLENBQWFDLGNBQTNCLEVBQTJDO0FBQ3pDUixNQUFBQSxRQUFRLENBQUNBLFFBQVEsQ0FBQ0csTUFBVCxHQUFrQixDQUFuQixDQUFSLElBQWlDRyxJQUFqQztBQUNELEtBRkQsTUFFTztBQUNMLFVBQUksS0FBS0MsT0FBTCxDQUFhRSxnQkFBakIsRUFBbUM7QUFDakNILFFBQUFBLElBQUksR0FBR0EsSUFBSSxDQUFDSSxJQUFMLEVBQVA7QUFDRDs7QUFDRFYsTUFBQUEsUUFBUSxDQUFDVyxJQUFULENBQWNMLElBQWQ7QUFDRDtBQUNGOztBQUVELFNBQU9OLFFBQVA7QUFDRCxDQXhCRDs7QUEwQk8sU0FBU1ksU0FBVCxDQUFtQkMsTUFBbkIsRUFBMkJDLE1BQTNCLEVBQW1DQyxRQUFuQyxFQUE2QztBQUFFLFNBQU9uQixRQUFRLENBQUNvQixJQUFULENBQWNILE1BQWQsRUFBc0JDLE1BQXRCLEVBQThCQyxRQUE5QixDQUFQO0FBQWlEOztBQUNoRyxTQUFTRSxnQkFBVCxDQUEwQkosTUFBMUIsRUFBa0NDLE1BQWxDLEVBQTBDQyxRQUExQyxFQUFvRDtBQUN6RCxNQUFJUixPQUFPO0FBQUc7QUFBQTtBQUFBOztBQUFBVztBQUFBQTtBQUFBQTtBQUFBQTtBQUFBQTtBQUFBQTtBQUFBO0FBQUEsR0FBZ0JILFFBQWhCLEVBQTBCO0FBQUNOLElBQUFBLGdCQUFnQixFQUFFO0FBQW5CLEdBQTFCLENBQWQ7QUFDQSxTQUFPYixRQUFRLENBQUNvQixJQUFULENBQWNILE1BQWQsRUFBc0JDLE1BQXRCLEVBQThCUCxPQUE5QixDQUFQO0FBQ0QiLCJzb3VyY2VzQ29udGVudCI6WyJpbXBvcnQgRGlmZiBmcm9tICcuL2Jhc2UnO1xuaW1wb3J0IHtnZW5lcmF0ZU9wdGlvbnN9IGZyb20gJy4uL3V0aWwvcGFyYW1zJztcblxuZXhwb3J0IGNvbnN0IGxpbmVEaWZmID0gbmV3IERpZmYoKTtcbmxpbmVEaWZmLnRva2VuaXplID0gZnVuY3Rpb24odmFsdWUpIHtcbiAgbGV0IHJldExpbmVzID0gW10sXG4gICAgICBsaW5lc0FuZE5ld2xpbmVzID0gdmFsdWUuc3BsaXQoLyhcXG58XFxyXFxuKS8pO1xuXG4gIC8vIElnbm9yZSB0aGUgZmluYWwgZW1wdHkgdG9rZW4gdGhhdCBvY2N1cnMgaWYgdGhlIHN0cmluZyBlbmRzIHdpdGggYSBuZXcgbGluZVxuICBpZiAoIWxpbmVzQW5kTmV3bGluZXNbbGluZXNBbmROZXdsaW5lcy5sZW5ndGggLSAxXSkge1xuICAgIGxpbmVzQW5kTmV3bGluZXMucG9wKCk7XG4gIH1cblxuICAvLyBNZXJnZSB0aGUgY29udGVudCBhbmQgbGluZSBzZXBhcmF0b3JzIGludG8gc2luZ2xlIHRva2Vuc1xuICBmb3IgKGxldCBpID0gMDsgaSA8IGxpbmVzQW5kTmV3bGluZXMubGVuZ3RoOyBpKyspIHtcbiAgICBsZXQgbGluZSA9IGxpbmVzQW5kTmV3bGluZXNbaV07XG5cbiAgICBpZiAoaSAlIDIgJiYgIXRoaXMub3B0aW9ucy5uZXdsaW5lSXNUb2tlbikge1xuICAgICAgcmV0TGluZXNbcmV0TGluZXMubGVuZ3RoIC0gMV0gKz0gbGluZTtcbiAgICB9IGVsc2Uge1xuICAgICAgaWYgKHRoaXMub3B0aW9ucy5pZ25vcmVXaGl0ZXNwYWNlKSB7XG4gICAgICAgIGxpbmUgPSBsaW5lLnRyaW0oKTtcbiAgICAgIH1cbiAgICAgIHJldExpbmVzLnB1c2gobGluZSk7XG4gICAgfVxuICB9XG5cbiAgcmV0dXJuIHJldExpbmVzO1xufTtcblxuZXhwb3J0IGZ1bmN0aW9uIGRpZmZMaW5lcyhvbGRTdHIsIG5ld1N0ciwgY2FsbGJhY2spIHsgcmV0dXJuIGxpbmVEaWZmLmRpZmYob2xkU3RyLCBuZXdTdHIsIGNhbGxiYWNrKTsgfVxuZXhwb3J0IGZ1bmN0aW9uIGRpZmZUcmltbWVkTGluZXMob2xkU3RyLCBuZXdTdHIsIGNhbGxiYWNrKSB7XG4gIGxldCBvcHRpb25zID0gZ2VuZXJhdGVPcHRpb25zKGNhbGxiYWNrLCB7aWdub3JlV2hpdGVzcGFjZTogdHJ1ZX0pO1xuICByZXR1cm4gbGluZURpZmYuZGlmZihvbGRTdHIsIG5ld1N0ciwgb3B0aW9ucyk7XG59XG4iXX0=
262-
263-
## HACK
264-
265-
- `inline.js:59` — technically, this function is not marked @internal so it's possible
266-
- `inline.js:462` — workaround node regression
26728

0 commit comments

Comments
 (0)