Remove unused files, drop prettier, fix biome issues by juniorbutyeah · Pull Request #303 · UseInterstellar/Interstellar-Astro

juniorbutyeah · 2026-01-30T07:00:02Z

Summary

Removed unused files (Obfuscated.astro, epoxy.d.ts, wisp.d.ts, prettier configs)
Dropped prettier in favor of biome for formatting
Fixed all biome lint issues in Browser.tsx, settings.ts, and index.ts
Excluded dist/ and public/assets/ from biome linting

Test plan

bun run typecheck passes
bun run check passes (biome)
Build succeeds

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

juniorbutyeah

lgtm

Copilot

Pull request overview

Copilot reviewed 16 out of 19 changed files in this pull request and generated 8 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-30T18:54:51Z

+          });
+  },
+);
+//# sourceMappingURL=scramjet.sync.js.map


This appears to be a generated/minified file (evidenced by the source map comment and the single-line format in the original). Generated files are typically excluded from formatting tools and should be regenerated from source rather than manually formatted. While biome exclusions have been added for public/assets in this PR, this file has already been formatted. Consider whether this file should be regenerated from its source instead, or if it should remain in its original minified form.

Suggested change

//# sourceMappingURL=scramjet.sync.js.map

// Source map: scramjet.sync.js.map

Copilot · 2026-01-30T18:54:51Z

+            }
+            for (; o < e.length; ) {
+              for (t = o, a = !1; l(); )
+                if ("," === (r = e.charAt(o))) {


The value assigned to r here is unused.

Copilot · 2026-01-30T18:54:52Z

+          return (t = (t = new String(t).toString()).replace(/url\(['"]?(.+?)['"]?\)/gm, (t, i) => {
+            const a = "rewrite" === e ? (0, n.Oy)(i.trim(), r) : (0, n.v2)(i.trim());
+            return t.replace(i, a);
+          })).replace(/@import\s+(url\s*?\(.{0,9999}?\)|['"].{0,9999}?['"]|.{0,9999}?)($|\s|;)/gm, (t, i) =>


The value assigned to t here is unused.

Copilot · 2026-01-30T18:54:52Z

+          return s("unrewrite", e);
+        }
+        function s(e, t, r) {
+          return (t = (t = new String(t).toString()).replace(/url\(['"]?(.+?)['"]?\)/gm, (t, i) => {


The value assigned to t here is unused.

Copilot · 2026-01-30T18:54:52Z

The value assigned to n here is unused.

Suggested change

var n, i;

r(8866), ((i = n || (n = {}))[(i.DISCONNECTED = 1)] = "DISCONNECTED"), (i[(i.PRECEDING = 2)] = "PRECEDING"), (i[(i.FOLLOWING = 4)] = "FOLLOWING"), (i[(i.CONTAINS = 8)] = "CONTAINS"), (i[(i.CONTAINED_BY = 16)] = "CONTAINED_BY");

var i;

r(8866), ((i = {})[(i.DISCONNECTED = 1)] = "DISCONNECTED"), (i[(i.PRECEDING = 2)] = "PRECEDING"), (i[(i.FOLLOWING = 4)] = "FOLLOWING"), (i[(i.CONTAINS = 8)] = "CONTAINS"), (i[(i.CONTAINED_BY = 16)] = "CONTAINED_BY");

Copilot · 2026-01-30T18:54:53Z

+          s,
+          o = r(747);
+        ((n = a || (a = {}))[(n.XML = 0)] = "XML"), (n[(n.HTML = 1)] = "HTML"), ((i = s || (s = {}))[(i.UTF8 = 0)] = "UTF8"), (i[(i.ASCII = 1)] = "ASCII"), (i[(i.Extensive = 2)] = "Extensive"), (i[(i.Attribute = 3)] = "Attribute"), (i[(i.Text = 4)] = "Text");


The value assigned to a here is unused.

Copilot · 2026-01-30T18:54:53Z

+          o = r(747);
+        ((n = a || (a = {}))[(n.XML = 0)] = "XML"), (n[(n.HTML = 1)] = "HTML"), ((i = s || (s = {}))[(i.UTF8 = 0)] = "UTF8"), (i[(i.ASCII = 1)] = "ASCII"), (i[(i.Extensive = 2)] = "Extensive"), (i[(i.Attribute = 3)] = "Attribute"), (i[(i.Text = 4)] = "Text");


The value assigned to s here is unused.

Suggested change

s,

o = r(747);

((n = a || (a = {}))[(n.XML = 0)] = "XML"), (n[(n.HTML = 1)] = "HTML"), ((i = s || (s = {}))[(i.UTF8 = 0)] = "UTF8"), (i[(i.ASCII = 1)] = "ASCII"), (i[(i.Extensive = 2)] = "Extensive"), (i[(i.Attribute = 3)] = "Attribute"), (i[(i.Text = 4)] = "Text");

o = r(747);

((n = a || (a = {}))[(n.XML = 0)] = "XML"), (n[(n.HTML = 1)] = "HTML"), ((i = i || (i = {}))[(i.UTF8 = 0)] = "UTF8"), (i[(i.ASCII = 1)] = "ASCII"), (i[(i.Extensive = 2)] = "Extensive"), (i[(i.Attribute = 3)] = "Attribute"), (i[(i.Text = 4)] = "Text");

Copilot · 2026-01-30T18:54:53Z

+                (i = t), (a = !0);
+                break;
+              }
+              !a && t.length > i.length && (i = t);


This negation always evaluates to true.

Copilot

Pull request overview

Copilot reviewed 16 out of 19 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

  const [favicons, setFavicons] = useState<Record<number, string>>({});
  const [bookmarks, setBookmarks] = useState<Array<{ Title: string; url: string; favicon?: string }>>([]);
-  const [proxyReadyTick, setProxyReadyTick] = useState(0);
+  const [_proxyReadyTick, setProxyReadyTick] = useState(0);


+    },
+    {
+      "includes": ["index.ts"],
+      "linter": {
+        "rules": {
+          "suspicious": {
+            "noExplicitAny": "off"
+          }
+        }
+      }


+addEventListener(
+  "message",
+  ({
+    data: {
+      sab: e,
+      args: [t, n, _s, r, o],
+      body: a,
+      headers: g,
+    },
+  }) => {
+    const i = new DataView(e),
+      l = new Uint8Array(e),
+      d = new XMLHttpRequest();
+    if (((d.responseType = "arraybuffer"), d.open(t, n, !0, r, o), g)) for (const [e, t] of Object.entries(g)) d.setRequestHeader(e, t);
+    d.send(a),
+      (d.onload = () => {
+        let t = 1;
+        i.setUint16(t, d.status), (t += 2);
+        const n = d.getAllResponseHeaders();
+        i.setUint32(t, n.length),
+          (t += 4),
+          e.byteLength < t + n.length && e.grow(t + n.length),
+          l.set(new TextEncoder().encode(n), t),
+          (t += n.length),
+          i.setUint32(t, d.response.byteLength),
+          (t += 4),
+          e.byteLength < t + d.response.byteLength && e.grow(t + d.response.byteLength),
+          l.set(new Uint8Array(d.response), t),
+          i.setUint8(0, 1);
+      }),
+      (d.ontimeout =
+        d.onerror =
+        d.onabort =
+          () => {
+            console.error("xhr failed"), i.setUint8(0, 1);
+          });
+  },
+);


xbubbo · 2026-04-17T13:35:42Z

junior said to merge this

Remove unused files, drop prettier, fix biome issues

9803b1a

Copilot AI review requested due to automatic review settings January 30, 2026 07:00

Copilot started reviewing on behalf of juniorbutyeah January 30, 2026 07:00 View session

Revert accidental bundle formatting

07071fc

Copilot AI reviewed Jan 30, 2026

View reviewed changes

juniorbutyeah commented Jan 30, 2026

View reviewed changes

juniorbutyeah requested a review from Copilot January 30, 2026 18:49

Copilot started reviewing on behalf of juniorbutyeah January 30, 2026 18:50 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

juniorbutyeah self-assigned this Jan 30, 2026

juniorbutyeah requested a review from xbubbo January 30, 2026 19:27

juniorbutyeah requested a review from Copilot March 14, 2026 20:26

Copilot started reviewing on behalf of juniorbutyeah March 14, 2026 20:26 View session

Copilot AI reviewed Mar 14, 2026

View reviewed changes

xbubbo closed this Apr 17, 2026

xbubbo reopened this Apr 17, 2026

xbubbo approved these changes Apr 17, 2026

View reviewed changes

xbubbo merged commit 3772564 into main Apr 17, 2026
14 of 15 checks passed

xbubbo deleted the cleanup-unused-files branch April 17, 2026 13:37

	//# sourceMappingURL=scramjet.sync.js.map
	// Source map: scramjet.sync.js.map

-        var n, i;
-        r(8866), ((i = n || (n = {}))[(i.DISCONNECTED = 1)] = "DISCONNECTED"), (i[(i.PRECEDING = 2)] = "PRECEDING"), (i[(i.FOLLOWING = 4)] = "FOLLOWING"), (i[(i.CONTAINS = 8)] = "CONTAINS"), (i[(i.CONTAINED_BY = 16)] = "CONTAINED_BY");
+        var i;
+        r(8866), ((i = {})[(i.DISCONNECTED = 1)] = "DISCONNECTED"), (i[(i.PRECEDING = 2)] = "PRECEDING"), (i[(i.FOLLOWING = 4)] = "FOLLOWING"), (i[(i.CONTAINS = 8)] = "CONTAINS"), (i[(i.CONTAINED_BY = 16)] = "CONTAINED_BY");

		o = r(747);
		((n = a \|\| (a = {}))[(n.XML = 0)] = "XML"), (n[(n.HTML = 1)] = "HTML"), ((i = s \|\| (s = {}))[(i.UTF8 = 0)] = "UTF8"), (i[(i.ASCII = 1)] = "ASCII"), (i[(i.Extensive = 2)] = "Extensive"), (i[(i.Attribute = 3)] = "Attribute"), (i[(i.Text = 4)] = "Text");

Conversation

juniorbutyeah commented Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

juniorbutyeah left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

xbubbo commented Apr 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

juniorbutyeah commented Jan 30, 2026 •

edited

Loading