Skip to content

Remove unused files, drop prettier, fix biome issues #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
juniorbutyeah wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Remove unused files, drop prettier, fix biome issues #303

juniorbutyeah wants to merge 2 commits into from
+5,648 −171

Conversation

@juniorbutyeah
Copy link
Collaborator

@juniorbutyeah juniorbutyeah commented Jan 30, 2026 •
edited
Loading

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 AI review requested due to automatic review settings January 30, 2026 07:00
Copilot AI reviewed Jan 30, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

juniorbutyeah
juniorbutyeah commented Jan 30, 2026
View reviewed changes
Copy link
Collaborator Author

@juniorbutyeah juniorbutyeah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@juniorbutyeah juniorbutyeah requested a review from Copilot January 30, 2026 18:49
Copilot AI reviewed Jan 30, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

public/assets/scramjet/scramjet.sync.js
});
},
);
//# sourceMappingURL=scramjet.sync.js.map
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
}
for (; o < e.length; ) {
for (t = o, a = !1; l(); )
if ("," === (r = e.charAt(o))) {
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value assigned to r here is unused.

Copilot uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
Comment on lines +2561 to +2564
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) =>
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value assigned to t here is unused.

Copilot uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
return s("unrewrite", e);
}
function s(e, t, r) {
return (t = (t = new String(t).toString()).replace(/url\(['"]?(.+?)['"]?\)/gm, (t, i) => {
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value assigned to t here is unused.

Copilot uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
Comment on lines +4396 to +4397
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");
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
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");
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value assigned to a here is unused.

Copilot uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
Comment on lines +4699 to +4701
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");
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 uses AI. Check for mistakes.
public/assets/scramjet/scramjet.all.js
(i = t), (a = !0);
break;
}
!a && t.length > i.length && (i = t);
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This negation always evaluates to true.

Copilot uses AI. Check for mistakes.
@juniorbutyeah juniorbutyeah self-assigned this Jan 30, 2026
@juniorbutyeah juniorbutyeah requested a review from xbubbo January 30, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@xbubbo xbubbo Awaiting requested review from xbubbo

At least 1 approving review is required to merge this pull request.

Assignees

@juniorbutyeah juniorbutyeah

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@juniorbutyeah