Skip to content

Commit

Permalink
Run Biome formatter (#1392)
Browse files Browse the repository at this point in the history 
* Run formatter

* Part 2
  • Loading branch information
@ethteck
ethteck authored Dec 10, 2024
1 parent a003f5b commit 4ef64cb
Show file tree
Hide file tree
Showing 140 changed files with 8,309 additions and 6,243 deletions.
5 changes: 0 additions & 5 deletions frontend/biome.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,5 @@
"useSortedClasses": "error"
}
}
},
"javascript": {
"formatter": {
"enabled": false
}
}
}
289 changes: 153 additions & 136 deletions frontend/next.config.js
View file
Original file line number Diff line number Diff line change
@@ -1,168 +1,185 @@
const { Compilation } = require("webpack")
const { execSync } = require("node:child_process")
const { config } = require("dotenv")
const { Compilation } = require("webpack");
const { execSync } = require("node:child_process");
const { config } = require("dotenv");

for (const envFile of [".env.local", ".env"]) {
config({ path: `../${envFile}` })
config({ path: `../${envFile}` });
}

const getEnvBool = (key, fallback = false) => {
const value = process.env[key]
const value = process.env[key];
if (value === "false" || value === "0" || value === "off") {
return false
return false;
}
if (value === "true" || value === "1" || value === "on") {
return true
return true;
}
return fallback
}
return fallback;
};

let git_hash
let git_hash;
try {
git_hash = execSync("git rev-parse HEAD", { stdio: "pipe" }).toString().trim()
git_hash = execSync("git rev-parse HEAD", { stdio: "pipe" })
.toString()
.trim();
} catch (error) {
console.log("Unable to get git hash, assume running inside Docker")
git_hash = "abc123"
console.log("Unable to get git hash, assume running inside Docker");
git_hash = "abc123";
}

const { withPlausibleProxy } = require("next-plausible")
const { withPlausibleProxy } = require("next-plausible");

const removeImports = require("next-remove-imports")({
//test: /node_modules([\s\S]*?)\.(tsx|ts|js|mjs|jsx)$/,
//matchImports: "\\.(less|css|scss|sass|styl)$"
})
});

const mediaUrl = new URL(process.env.MEDIA_URL ?? "http://localhost")
const mediaUrl = new URL(process.env.MEDIA_URL ?? "http://localhost");

let app = withPlausibleProxy({
customDomain: "https://stats.decomp.me",
})(removeImports({
async redirects() {
return [
{
source: "/scratch",
destination: "/",
permanent: true,
},
{
source: "/scratch/new",
destination: "/new",
permanent: true,
},
{
source: "/settings",
destination: "/settings/account",
permanent: false,
},
]
},
async rewrites() {
return []
},
async headers() {
return [
{
source: "/(.*)", // all routes
headers: [
{
key: "X-DNS-Prefetch-Control",
value: "on",
},
{
key: "Cross-Origin-Opener-Policy",
value: "same-origin",
},
{
key: "Cross-Origin-Embedder-Policy",
value: "require-corp",
},
],
},
]
},
webpack(config) {
config.module.rules.push({
test: /\.svg$/,
use: ["@svgr/webpack"],
})
})(
removeImports({
async redirects() {
return [
{
source: "/scratch",
destination: "/",
permanent: true,
},
{
source: "/scratch/new",
destination: "/new",
permanent: true,
},
{
source: "/settings",
destination: "/settings/account",
permanent: false,
},
];
},
async rewrites() {
return [];
},
async headers() {
return [
{
source: "/(.*)", // all routes
headers: [
{
key: "X-DNS-Prefetch-Control",
value: "on",
},
{
key: "Cross-Origin-Opener-Policy",
value: "same-origin",
},
{
key: "Cross-Origin-Embedder-Policy",
value: "require-corp",
},
],
},
];
},
webpack(config) {
config.module.rules.push({
test: /\.svg$/,
use: ["@svgr/webpack"],
});

// @open-rpc/client-js brings in some dependencies which, in turn, have optional dependencies.
// This confuses the heck out of webpack, so tell it should just sub in a CommonJS-style "require" statement
// instead (which will fail and trigger the fallback at runtime)
// https://stackoverflow.com/questions/58697934/webpack-how-do-you-require-an-optional-dependency-in-bundle-saslprep
config.externals.push({
"encoding": "commonjs encoding",
"bufferutil": "commonjs bufferutil",
"utf-8-validate": "commonjs utf-8-validate",
})
// @open-rpc/client-js brings in some dependencies which, in turn, have optional dependencies.
// This confuses the heck out of webpack, so tell it should just sub in a CommonJS-style "require" statement
// instead (which will fail and trigger the fallback at runtime)
// https://stackoverflow.com/questions/58697934/webpack-how-do-you-require-an-optional-dependency-in-bundle-saslprep
config.externals.push({
encoding: "commonjs encoding",
bufferutil: "commonjs bufferutil",
"utf-8-validate": "commonjs utf-8-validate",
});

// All of the vscode-* packages (jsonrpc, languageserver-protocol, etc.) are distributed as UMD modules.
// This also leaves webpack with no idea how to handle require statements.
// umd-compat-loader strips away the hedaer UMD adds to allow browsers to parse ES modules
// and just treats the importee as an ES module.
config.module.rules.push({
"test": /node_modules[\\|/](vscode-.*)/,
"use": {
"loader": "umd-compat-loader",
},
})
// All of the vscode-* packages (jsonrpc, languageserver-protocol, etc.) are distributed as UMD modules.
// This also leaves webpack with no idea how to handle require statements.
// umd-compat-loader strips away the hedaer UMD adds to allow browsers to parse ES modules
// and just treats the importee as an ES module.
config.module.rules.push({
test: /node_modules[\\|/](vscode-.*)/,
use: {
loader: "umd-compat-loader",
},
});

// XXX: Terser/SWC currently breaks while minifying objdiff's ESM worker in the static directory because
// it uses `import.meta.url`. next.js provides no way to control this behavior, of course, so we'll
// hook into the optimization stage and mark assets in `static/media/` as already minified.
// https://github.com/vercel/next.js/issues/33914
// https://github.com/vercel/next.js/discussions/61549
config.optimization.minimizer.unshift({
apply(compiler) {
const pluginName = "SkipWorkerMinify"
compiler.hooks.thisCompilation.tap(pluginName, compilation=>{
compilation.hooks.processAssets.tap({
name: pluginName,
stage: Compilation.PROCESS_ASSETS_STAGE_OPTIMIZE_SIZE,
}, assets=>{
for (const assetName in assets) {
if (/^static\/media\//.test(assetName)) {
compilation.updateAsset(assetName, assets[assetName], {
minimized: true,
})
}
}
})
})
},
})
// XXX: Terser/SWC currently breaks while minifying objdiff's ESM worker in the static directory because
// it uses `import.meta.url`. next.js provides no way to control this behavior, of course, so we'll
// hook into the optimization stage and mark assets in `static/media/` as already minified.
// https://github.com/vercel/next.js/issues/33914
// https://github.com/vercel/next.js/discussions/61549
config.optimization.minimizer.unshift({
apply(compiler) {
const pluginName = "SkipWorkerMinify";
compiler.hooks.thisCompilation.tap(
pluginName,
(compilation) => {
compilation.hooks.processAssets.tap(
{
name: pluginName,
stage: Compilation.PROCESS_ASSETS_STAGE_OPTIMIZE_SIZE,
},
(assets) => {
for (const assetName in assets) {
if (
/^static\/media\//.test(assetName)
) {
compilation.updateAsset(
assetName,
assets[assetName],
{
minimized: true,
},
);
}
}
},
);
},
);
},
});

return config
},
images: {
remotePatterns: [{
// Expected 'http' | 'https', received 'http:' at "images.remotePatterns[0].protocol"
protocol: mediaUrl.protocol.replace(":", ""),
hostname: mediaUrl.hostname,
port: mediaUrl.port,
pathname: "/**",
return config;
},
images: {
remotePatterns: [
{
// Expected 'http' | 'https', received 'http:' at "images.remotePatterns[0].protocol"
protocol: mediaUrl.protocol.replace(":", ""),
hostname: mediaUrl.hostname,
port: mediaUrl.port,
pathname: "/**",
},
{
protocol: "https",
hostname: "avatars.githubusercontent.com",
port: "",
pathname: "/**",
},
],
unoptimized: !getEnvBool("FRONTEND_USE_IMAGE_PROXY"),
},
{
protocol: "https",
hostname: "avatars.githubusercontent.com",
port: "",
pathname: "/**",
swcMinify: true,
env: {
// XXX: don't need 'NEXT_PUBLIC_' prefix here; we could just use 'API_BASE' and 'GITHUB_CLIENT_ID'
// See note at top of https://nextjs.org/docs/api-reference/next.config.js/environment-variables for more information
NEXT_PUBLIC_API_BASE: process.env.API_BASE,
NEXT_PUBLIC_GITHUB_CLIENT_ID: process.env.GITHUB_CLIENT_ID,
NEXT_PUBLIC_COMMIT_HASH: git_hash,
},
],
unoptimized: !getEnvBool("FRONTEND_USE_IMAGE_PROXY"),
},
swcMinify: true,
env: {
// XXX: don't need 'NEXT_PUBLIC_' prefix here; we could just use 'API_BASE' and 'GITHUB_CLIENT_ID'
// See note at top of https://nextjs.org/docs/api-reference/next.config.js/environment-variables for more information
NEXT_PUBLIC_API_BASE: process.env.API_BASE,
NEXT_PUBLIC_GITHUB_CLIENT_ID: process.env.GITHUB_CLIENT_ID,
NEXT_PUBLIC_COMMIT_HASH: git_hash,
},
}))
}),
);

if (process.env.ANALYZE === "true") {
app = require("@next/bundle-analyzer")(app)
app = require("@next/bundle-analyzer")(app);
}

module.exports = app
module.exports = app;
8 changes: 2 additions & 6 deletions frontend/postcss.config.js
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
module.exports = {
plugins: [
"tailwindcss",
"autoprefixer",
"cssnano",
],
}
plugins: ["tailwindcss", "autoprefixer", "cssnano"],
};
Loading

0 comments on commit 4ef64cb

Please sign in to comment.