refactor: 🚨 Remove unnecessary async keywords

This commit is contained in:
Jesse Wierzbinski 2024-10-03 11:43:16 +02:00
parent 132bec4d5b
commit b1d8595a7c
No known key found for this signature in database
10 changed files with 38 additions and 39 deletions

View file

@ -81,7 +81,7 @@ export default apiRoute((app) =>
);
return context.json(
await Promise.all(favourites.map(async (note) => note.toApi(user))),
await Promise.all(favourites.map((note) => note.toApi(user))),
200,
{
Link: link,

View file

@ -37,7 +37,7 @@ const route = createRoute({
});
export default apiRoute((app) =>
app.openapi(route, async (context) => {
app.openapi(route, (context) => {
return context.json(
config.signups.rules.map((rule, index) => ({
id: String(index),

View file

@ -93,7 +93,7 @@ export default apiRoute((app) =>
);
return context.json(
await Promise.all(objects.map(async (note) => note.toApi(user))),
await Promise.all(objects.map((note) => note.toApi(user))),
200,
{
Link: link,

View file

@ -105,7 +105,7 @@ export default apiRoute((app) =>
);
return context.json(
await Promise.all(objects.map(async (note) => note.toApi(user))),
await Promise.all(objects.map((note) => note.toApi(user))),
200,
{
Link: link,

View file

@ -11,7 +11,6 @@ export class BlurhashPreprocessor implements MediaPreprocessor {
const metadata = await sharp(arrayBuffer).metadata();
const blurhash = await new Promise<string | null>((resolve) => {
(async () =>
sharp(arrayBuffer)
.raw()
.ensureAlpha()
@ -34,7 +33,7 @@ export class BlurhashPreprocessor implements MediaPreprocessor {
} catch {
resolve(null);
}
}))();
});
});
return { file, blurhash };

View file

@ -80,7 +80,7 @@ describe("PluginLoader", () => {
test("parseManifestFile should parse JSON manifest", async () => {
const manifestContent = { name: "test-plugin" };
Bun.file = jest.fn().mockReturnValue({
text: async () => JSON.stringify(manifestContent),
text: () => Promise.resolve(JSON.stringify(manifestContent)),
});
// biome-ignore lint/complexity/useLiteralKeys: Private method
@ -111,7 +111,7 @@ describe("PluginLoader", () => {
};
mockReaddir.mockResolvedValue(["manifest.json"]);
Bun.file = jest.fn().mockReturnValue({
text: async () => JSON.stringify(manifestContent),
text: () => Promise.resolve(JSON.stringify(manifestContent)),
});
manifestSchema.safeParseAsync = jest.fn().mockResolvedValue({
success: true,
@ -141,7 +141,7 @@ describe("PluginLoader", () => {
};
mockReaddir.mockResolvedValue(["manifest.json"]);
Bun.file = jest.fn().mockReturnValue({
text: async () => JSON.stringify(manifestContent),
text: () => Promise.resolve(JSON.stringify(manifestContent)),
});
manifestSchema.safeParseAsync = jest.fn().mockResolvedValue({
success: false,
@ -188,7 +188,7 @@ describe("PluginLoader", () => {
])
.mockResolvedValue(["manifest.json", "index.ts"]);
Bun.file = jest.fn().mockReturnValue({
text: async () => JSON.stringify(manifestContent),
text: () => Promise.resolve(JSON.stringify(manifestContent)),
});
manifestSchema.safeParseAsync = jest.fn().mockResolvedValue({
success: true,

View file

@ -149,7 +149,7 @@ export class SonicSearchManager {
* @param n Batch number
* @param batchSize Size of the batch
*/
private async getNthDatabaseAccountBatch(
private getNthDatabaseAccountBatch(
n: number,
batchSize = 1000,
): Promise<Record<string, string | Date>[]> {
@ -172,7 +172,7 @@ export class SonicSearchManager {
* @param n Batch number
* @param batchSize Size of the batch
*/
private async getNthDatabaseStatusBatch(
private getNthDatabaseStatusBatch(
n: number,
batchSize = 1000,
): Promise<Record<string, string | Date>[]> {

View file

@ -172,10 +172,10 @@ export class Emoji extends BaseInterface<typeof Emojis, EmojiWithInstance> {
* @param text The text to parse
* @returns An array of emojis
*/
public static async parseFromText(text: string): Promise<Emoji[]> {
public static parseFromText(text: string): Promise<Emoji[]> {
const matches = text.match(emojiValidatorWithColons);
if (!matches || matches.length === 0) {
return [];
return Promise.resolve([]);
}
return Emoji.manyFromSql(

View file

@ -395,8 +395,8 @@ export const setContextFormDataToObject = (
setTo: object,
): Context => {
context.req.bodyCache.json = setTo;
context.req.parseBody = async () => context.req.bodyCache.json;
context.req.json = async () => context.req.bodyCache.json;
context.req.parseBody = () => Promise.resolve(context.req.bodyCache.json);
context.req.json = () => Promise.resolve(context.req.bodyCache.json);
return context;
};

View file

@ -47,11 +47,11 @@ export const urlToContentFormat = (url?: string): ContentFormat | null => {
};
};
export const mimeLookup = async (url: string) => {
export const mimeLookup = (url: string): Promise<string> => {
const naiveLookup = lookup(url.replace(new URL(url).search, ""));
if (naiveLookup) {
return naiveLookup;
return Promise.resolve(naiveLookup);
}
const fetchLookup = fetch(url, {