#👾-core-development

1 messages · Page 198 of 1

charred monolithBOT
median rapids
#

there's your answer

frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

charred monolithBOT
#

Discord Account

No response

What happens when the bug or crash occurs?

when clicking a persons profile whom has lots of roles, the profile just gets stuck, you're not able to scroll up or down, leaving to have to use the right click menu if you want to try and add them as a friend and such.
image

What is the expected behaviour?

I expect ShowAllRoles to handle this kind of probl...

shell shuttle
#

Link isn't real

#

it's <a>

median rapids
#

I thought it is

#

a discord component

jagged reef
#

Shouldn't it be <Lionk href="" />

median rapids
#

look 2 lines after

jagged reef
#

I have the reading capability of a plugin request opener/j

crude hearth
#

For dynamic sites I think Link is being used, tho it might not make sense in this case

crude hearth
charred monolithBOT
sterile oak
austere talon
#

the pinned post in question

sterile oak
#

she said to open a modmail to get access

#

This means that your repository needs to have its index.ts file in the root. One plugin per repo! why do I feel targeted SKULL_SKELETON

charred monolithBOT
austere talon
#

wdym?

#

you always had to have your plugins like this

charred monolithBOT
austere talon
#

how was it supposed to have worked before

#

it only checks userplugins/plugin/index.ts

not userplugins/nico/plugins/index.ts

#

that's fucking awful

#

vencord could support nested folders so you can clone entire repos

#

but also people should not have to install all your plugins just to use one

#

vencord might update userplugins for you

#

the entire reason the new rule is there so that all userplugins are git repos you can pull

#

instead of downloading files manually

#

in the future i might make vencord ignore userplugins that are not git repositories

#

to enforce proper installation that allows updating them

#

with some way to still load ur own plugins without a git repo

verbal pumice
#

oh i should probably read first

fallen vale
granite sky
verbal pumice
#

but then vscode gets all pissed

#

i think

austere talon
#

nah

#

i wanted to use submodules for userplugins

#

but submodules don't work in gitignored folders

turbid hatch
#

@austere talon i just thought

#

how do we do localisation for userplugins

#

or is that a "we're not doing it"

#

cuz atm everything is in one folder at the root

#

plugins could probably have a locale folder now that i think about it

#

i think i can make i18n ally support that

austere talon
#

we don't

turbid hatch
#

so only first parties get localisation?

austere talon
#

why in the world would anyone localise their tp plugin

#

not happening lmao

#

powercord allowed plugins to localise and there were only like 2 devs using it and they just did one locale (their native language)

turbid hatch
#

fair enough xd

#

it wouldnt get added to crowdin anyway so its useless

charred monolithBOT
turbid hatch
#

that was messy :<

charred monolithBOT
dusk blaze
#

(maybe not i just quick looked from mobile app)

rugged spire
#

mods dead

#

(i know nobody here can see it)

dusk blaze
#

why

rugged spire
#

its a modmail ticket

#

trol

#

asking about those things

dusk blaze
#

oh

#

I'll just make org on my forgejo instance

rugged spire
#

git branches should be fine IMO but I have asked about it

#

i will add you to ticket

#

horror

dusk blaze
#

afaik you can still invite

rugged spire
#

is there a fucking permission required to add to ticket

rugged spire
#

discord insane

dusk blaze
desert cosmos
rugged spire
#

me thinks the idea of plugin vendors are still a good idea now

fallen vale
dusk blaze
#

submits review
PR immediately closed

rugged spire
#

dom manipulation is husk husk husk husk husk husk

fallen vale
#

why point them to the rules then close it

dusk blaze
#

they can remake pr when they don't use deranged practices imo

austere talon
rugged spire
#

and see

#

I tried

#

and discord was like nuh uh

austere talon
#

actually only mods can yeah

turbid hatch
#
  • it's clear they didn't read the rules in the first place
dusk blaze
#

they opened 3 prs
3 rule breaking

rugged spire
#

nvm

austere talon
#

BD has put DOM brainrot into people's head

#

i hate it

dusk blaze
#

yop
so sad

rugged spire
#

the answer to "Should I be using DOM Manipulation in my Vencord plugin?" is
Short answer: No
Long answer: 99% of cases, no, however Vencord's core does it for things like CSS. If you MUST, do it within React with a ref for very very niche cases

#

i fixed it

charred monolithBOT
#

This pr has been reopened because vendicated closed the other pr for almost no reason that makes sense (#2648)

Just so you know vendicated, we are using github.io and we cannot do anything with that!

Also, why was the PR closed? You’ve mentioned the plugin rules but that didn’t explain much why it got closed.

Anyways,
This plugin allows you to add custom profile effects on your Discord profile
You just simply create an effect, submit it and wait for approval

Everything a...

spice python
glass jasper
spice python
#

love how they pretend to not know why it was closed and also know why it was closed

charred monolithBOT
glass jasper
fallen vale
#

Popular services like [...] GitHub are fine
??
genuinely confused

rugged spire
#

aka: come back when you have a reputation

spice python
austere talon
#

they could put arbitrary urls in that json

rugged spire
#

do you mean

#

arbitrary code or just "what if secretly ip logger"

#

probably latter

austere talon
#

the latter

glass jasper
austere talon
#

unless they find a vulnerability in chromes image parser

fallen vale
#

they didnt even format it smh

rugged spire
dusk blaze
rugged spire
#

the one thing I hate about decor is that there's no "unlisted" privacy option for you to steal other people's decorations

dusk blaze
#

blackbox is usrbg

rugged spire
#

im dum

#

my brain has been cooked for the past 4 days

dusk blaze
#

or do they already have

rugged spire
#

they already have

glass jasper
# glass jasper i just realized what you mean,

also “serstars” is a co-dev on almost all them makes me suspicious on the fact that they prolly didn’t actually write any of them themselves or that’s their alt-account or somethin

dusk blaze
#

idk likely both collaborate on plugins

fallen vale
grave mangoBOT
dusk blaze
#

also
@austere talon is putting links to your donation site allowed in plugin settings, if this plugin requires a custom server that you operate yourself (not planning to do that just wondering)

dusk blaze
#

reviuwerdb also does

rugged spire
#

huh it doesnt

dusk blaze
#

plugin settings

fallen vale
rugged spire
#

i meant decor

#

rdb does

rugged spire
austere talon
glass jasper
dusk blaze
spice python
#

actually it was someone elses pr

glass jasper
spice python
#

why were they even added to authors

#

they state it was an edit, however I dont see anything in common

glass jasper
#

for that badge i’m guessing

spice python
#

but it wasnt even his pr

#

the other person added them to authors of their pr

glass jasper
spice python
#

it says it's an edit of their pr, but I dont see anything in common really

glass jasper
#

honestly i’m just going to leave it on what Cynosphere said

spice python
#

petition to block people who repeatedly submit prs that are obviously just to get the contributor badge (also pretty sure I saw this when that pr was opened)

#

one sec let me try to find that

rugged spire
#

vencord has the ability to keep someone as a dev but hide the badge trolley

spice python
#

why did this get closed :( #2362

glass jasper
spice python
#

yes

dusk blaze
fallen vale
spice python
fallen vale
#

but they arent like,, manually given to individual users.

#

thats how i took that

spice python
#

I take automatically as automatically giving it to them once their pr gets merged, even if they didnt add themself to the devs constant in the pr

fallen vale
spice python
#

nvm it's third_party

spice python
austere talon
glass jasper
# spice python me when no updates

that’s why dev edition because you get the update a few month before other people and you have more control over it what you can with it

austere talon
#

literally comes preinstalled with vencord

#

and android too

glass jasper
austere talon
spice python
#

isnt android just linux

austere talon
#

android is based on linux yes

glass jasper
rugged spire
#

i want to add vendor to vencord

spice python
#

wth is /vendor/libdrm.so

glass jasper
spice python
#

is any of this malware

dusk blaze
granite sky
rugged spire
#

guh

#

why the FUCK does this check exist

#

called from 915885

#

this component

hexed star
fallen vale
glass jasper
#

as the great cyno once said

hexed star
# glass jasper

btw at that time I didn't know there's even badge for it, I thought u'll just have ur avatar in plugin settings KEKL

rugged spire
#

vencord experience

charred monolithBOT
#

Before I explain what this does, here's what it doesn't do (for people who stumble upon this PR)

This does not and cannot bypass permissions for that menu (I have tried before)

This adds a few more options to ShowHiddenThings, related to the Members page, visible as a channel linked in the sidebar, and in the settings. Unfortunately, Discord adds arbitrary restrictions based upon if your server is a "community" or not to decide where the page is visible.

Options

Show Members ...

rugged spire
#

yes I know titlegore much

still wasp
#

@rugged spire i think it's possible to do a similar thing with the roles page, you can view them and their permissions regardless of permission

#

I might investigate it

rugged spire
still wasp
#

Oh word

rugged spire
#

fun fact

#

I was gonna originally make the redirect to channel page button always shown

#

because I (naively) assumed the redirect icon was its own button to do that

sterile oak
#

my adhd is not letting me forget this:

grave mangoBOT
sterile oak
#

I can't stop thinking about correcting those. WHY THEY ARE ARRAYS IN THE FIRST PLACE

grave mangoBOT
rugged spire
#

you can always just uh shiddohwell

verbal pumice
#

inb4 added themselves to devs

sterile oak
verbal pumice
rugged spire
#

insane™️

sterile oak
#

I am going to watch netflix blobcatcozy

long stream
#

can we just block him already uwaaaaa

fallen vale
#

the comedy tho

glass jasper
long stream
rugged spire
rancid sigil
#

wild

fallen vale
rugged spire
#

true

#

although like I might've said

#

auto locking those

#

is a stupid idea

fallen vale
#

how come?/genq

rugged spire
#

if you don't lock them they can complain there

fallen vale
#

fair

still wasp
fossil inlet
glass jasper
shell shuttle
#

what did they do

rugged spire
fossil inlet
austere talon
#

discord update detector

grave mangoBOT
#

VencordTab.tsx: Line 96

note: "You need a theme that supports transparency or this will do nothing. WILL STOP THE WINDOW FROM BEING RESIZABLE!! Requires a full restart"
rugged spire
#

you are right

#

because windows is hollow

#

yeah?

#

sounds like a good idea

charred monolithBOT
charred monolithBOT
#

It's at least been broken since the Jun 18 Doomsday, and nothing is logged to the console during or after the decryption attempt. I don't know enough to confidently debug the script to find where the problem is. Using StegCloak in Node REPL decrypts the message as intended.

127 switzrr@switz-nyarch ~ % node
Welcome to Node.js v20.15.0.
Type ".help" for more information.
> const StegCloak = require('stegcloak');
undefined
> const stegcloak = new StegCloak(true, false);
undefined
>...
charred monolithBOT
verbal pumice
# charred monolith

given it took so long for someone to bother fixing this i wonder if we should just remove the plugin tbh

still wasp
granite sky
#

At least 2 people

still wasp
#

Yeah it's really useless

#

Interesting proof of concept but when would you use that???

dusk blaze
#

basic form of hiding activity

fallen vale
#
  1. make a server
#
  1. dont invite anyone
#
  1. profit
dusk blaze
#

physical access joined chat

fallen vale
#

how to deal with that

#
  1. turn off computer
#
  1. profit
glass jasper
#

how to deal with that

  1. remotely turn on the computer
  2. profit
fallen vale
#

how the ever living fuck do you remotely turn on and unlock a computer you dont own

glass jasper
#

exactly they never expect it

fallen vale
#

💀

glass jasper
#

anyways, it’s pretty easy to remotely turn on a PC. I do it quite often when I’m a over the road with my computer

fallen vale
#

and if someone else has access to your computer, you obviously dont care that much about hiding stuff.

glass jasper
fallen vale
#

my entire point though is that its your computer
you cant argue against my point by saying "but just ignore that tho"

rugged spire
#

WHAT DID YOU COOK

austere talon
rugged spire
#

why did you force push

#

stop it nyabom

austere talon
#

no

rugged spire
#

my bad i didnt cook today

#

basically i did the stupid

desert cosmos
#

insane

charred monolithBOT
#
[Vendicated/Vencord] New branch created: edit-users
shell shuttle
dusk blaze
#

what is vee cooking 🔥

austere talon
#

he takes 2.5k lines to do something u can do in 100 lines

rugged spire
#

what the fuck

austere talon
#

but still nowhere near 2.5k

rugged spire
#

true

glass jasper
sterile oak
fallen vale
rugged spire
austere talon
still wasp
#

It can have some uses

austere talon
#

my motivation for making it is @opaque pewter

still wasp
#

Wow that is

#

Unsettling

fallen vale
# austere talon

why the fuck do you have 75 mutual friends with a single person

still wasp
rugged spire
sterile oak
still wasp
#

I don't even have 75 friends total NAH

rugged spire
#

this my best I can find within my friends I think

fallen vale
still wasp
#

My best is like 12 😭

#

10/10 description

fallen vale
charred monolithBOT
rugged spire
#

That is a description indeed.

still wasp
#

Whom

austere talon
#

when the wip plugin doesn't have a description yet

still wasp
#

Shame on you

#

You will describe immediately

#

DON'T :THAT: REACDT ME IT WAS AQ JOKE

fallen vale
#

get that'd loser

hexed star
fossil inlet
#

who even is that

#

how do you have that many

frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

long heart
#

fire venware

#

no errors

still wasp
#

Venware my beloved

charred monolithBOT
sharp moss
#

what did i just see

fallen vale
#

admin abuse

#

./s

limber skiff
#

rm -rf message by github

sharp moss
#

GitHub Commit

charred monolithBOT
charred monolithBOT
odd heath
#

pronoundb still needs to be fixed for new profiles

kindred estuary
#

yeh I kinda prefered the old too

fallen vale
#

it scaled weirdly tho with fonts

#

that was my only issue with it

kindred estuary
#

I never had issues with the old

sterile oak
#

combine the two, I am thinking of gray button with text, and the icon aside the text

fossil inlet
#

is this the new vencord settings

#

if so, horror

#

AAAA

#

yea, that is terrible ui

charred monolithBOT
fossil inlet
#

i want to see what they do without hovering

#

normal buttons are fine as long as there's text

#

but this is really bad

#

also it clashes with the buttons on the bottom

charred monolithBOT
fossil inlet
#

yea, but still, thats four other buttons that just look out of place and i have no idea what they do

#

yea, i saw that

fallen vale
fossil inlet
#

just give the buttons labels

austere talon
#

what

fossil inlet
fallen vale
austere talon
#

if you don't like the new quick actions, then suggest an alternative cause the old ones also were not it

austere talon
#

maybe like this

fossil inlet
#

that would fix it

sterile oak
charred monolithBOT
sterile oak
austere talon
#

i should cherry pick those webpack improvements onto dev

still wasp
#

That is

#

Definitely something

austere talon
#

happy?

fallen vale
austere talon
#

was surprisingly hard to make that look good

#

like be responsive

still wasp
#

That is so good

charred monolithBOT
royal panther
glass jasper
glass jasper
austere talon
austere talon
#

why do you think i used icons

glass jasper
still wasp
austere talon
#

no

#

button count depends on build type

glass jasper
#

just a button that does nothing at all

dapper crest
limber skiff
glass jasper
#

makes it look better

austere talon
#

looks awesome

dapper crest
glass jasper
#

still would make it look better tho

dapper crest
glass jasper
austere talon
#

feel free to send a pull request 😉

charred monolithBOT
dapper crest
austere talon
#

no

dapper crest
#

OK then I will

austere talon
#

wait how the fuck does this work

dapper crest
#

Can edit client theme be added to quick actions?

austere talon
#

no

#

it's already in themes tab

dapper crest
#

Ah

glass jasper
#

nvm

#

it’s above

desert cosmos
desert cosmos
# austere talon happy?

stupid question but would it be possible to make that accessible to plugins similar to toolboxActions?

#

would be fun

dapper crest
#

am I dumb, there is no error in logs

austere talon
charred monolithBOT
dapper crest
#

LOOKS SO MUCH BETTER

.vc-settings-quickActions-card {
  display: flex;
  flex-wrap: wrap;
}
glass jasper
# austere talon

i like that but the edit clienttheme button need to be in the middle

glass jasper
dapper crest
#

one second

charred monolithBOT
austere talon
#

okay im done now sorrykirino

austere talon
#

i think it looks bad if they're inconsistently aligned

glass jasper
woeful sleet
#

Can you force push an empty commit? just to annoy some people

austere talon
charred monolithBOT
austere talon
#

thats too bad

limber skiff
#

yeah I saw

austere talon
#

cleaned up webpack methods code and made some more functions able to search by regex and use canonicalizeMatch

#

i needed it for mapMangledModule

limber skiff
#

I'm gonna add the regex functionality later

#

to my refactor

#

didnt think we would ever need it lmao

#

was an array of code not enough?

austere talon
#

well

#
318661: function(e, t, n) {
    "use strict";
    n.d(t, {
        Of: function() {
            return c
        },
        ZP: function() {
            return l
        }
    }),
    n(47120),
    n(470079);
    var r = n(442837)
      , i = n(594174)
      , a = n(251625)
      , o = n(255564)
      , s = n(621853);
    function l(e, t) {
        return (0,
        r.e7)([i.default, s.Z], ()=>c(e, t, [i.default, s.Z]))
    }
    n(484459);
    let u = (0,
    a.oH)((e,t)=>new o.Z(e,t));
    function c(e, t) {
        let[n,r] = arguments.length > 2 && void 0 !== arguments[2] ? arguments[2] : [i.default, s.Z]
          , a = n.getUser(e)
          , o = r.getUserProfile(e)
          , l = r.getGuildMemberProfile(e, t);
        return null == a || null == o ? null : u(o, l)
    }
},
#

not very robust

#

also technically regex should be faster

limber skiff
#

hmm no idea of it's not faster

#

string searches are super fast

#

I'm not against adding the support for regex but I would still use array of strings when possible

limber skiff
#

I thought it was one for a second

#

makes sense

austere talon
#

how is it so fast

fallen vale
#

uwu

austere talon
#

oops

verbal pumice
#

average github pr

charred monolithBOT
fallen vale
desert cosmos
still wasp
#

The pipeline

rugged spire
#

quick find some funny example to reply with that you SHOULD do

charred monolithBOT
nimble plaza
#

vp applem

nimble pendantBOT
charred monolithBOT
long stream
#

@nimble plaza fyi the plugin is hidden if you're not on macOS

#

you should include that in your reply blobcatcozy

nimble plaza
#

oh

long stream
nimble plaza
#

updated

#

isnt there apple music for windows

#

why make it exclusively macos

long stream
#

it uses a mac-specific api im pretty sure

#

oh applescript

nimble plaza
#

oh

grave mangoBOT
# long stream https://github.com/Vendicated/Vencord/blob/5c05443f454dd04288108330875f437889dc6...

native.ts: Lines 94-113

const playerState = await applescript(['tell application "Music"', "get player state", "end tell"])
    .then(out => out.trim());
if (playerState !== "playing") return null;

const playerPosition = await applescript(['tell application "Music"', "get player position", "end tell"])
    .then(text => Number.parseFloat(text.trim()));

const stdout = await applescript([
    'set output to ""',
    'tell application "Music"',
    "set t_id to database id of current track",
    "set t_name to name of current track",
    "set t_album to album of current track",
    "set t_artist to artist of current track",
    "set t_duration to duration of current track",
    'set output to "" & t_id & "\\n" & t_name & "\\n" & t_album & "\\n" & t_artist & "\\n" & t_duration',
    "end tell",
    "return output"
]);

long stream
#

pretty sure the windows client doesn't let you do any of this

limber skiff
#

lint is not passing btw

charred monolithBOT
frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

lone wren
#

W

charred monolithBOT
#

bda0e18 browser: increase minimum browser versions - Vendicated
169edcb improve support helper - Vendicated
086c31c new plugin ShowAllRoles: show all roles in simp... - Luna-devv
2d570a5 friendsSince: add support for new profiles (#2623) - Chicken
8a7c0d7 WebContextMenus: implement context menu for tex... - Masterjoona

limber skiff
#

fixed lint

charred monolithBOT
#
[Vendicated/Vencord] branch deleted: reviewdb-simplified-profile
#
[Vendicated/Vencord] branch deleted: permviewer-simplified-profiles
frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

charred monolithBOT
austere talon
#

but that's fine too

austere talon
#

maybe .dev plugin?

limber skiff
#

thinking better wouldnt it be better to have this enabled by default

#

less errors for users to send us screenshots

#

with this disabled

limber skiff
#

WHY

austere talon
limber skiff
#

like the font one

austere talon
#

i swear this bot hates you

#

it only cries for u

limber skiff
#

ikr

#

and this one was so random

#

meh I will just make the plugin .dev only

charred monolithBOT
austere talon
#

can u add something that stops the spotify store from logging ur spotify name and access token

limber skiff
#

good idea

austere talon
#

both are bad

limber skiff
#

an option or default behavior?

austere talon
#

i love when people share debug log and i have to tell them to deauthorise discord from spotify cause the screenshot included their spotify access token 😭

austere talon
#

maybe option

#

but default enabled

limber skiff
#

where even is the token?

#

I see my id I think

limber skiff
#

I KNEW IT

limber skiff
limber skiff
#

found it

#

this has too many logs with the account id, I will just disable the logger

austere talon
#

it logs username and token

limber skiff
#

I know

#

oh wait

#

username?

austere talon
#

ye somewhere

#

Devices updated for {name}

#

oh it's account id

#

why is my account id my username

limber skiff
#

I disabled it completely regardless

charred monolithBOT
austere talon
#

😭

#

third log

limber skiff
#

god

austere talon
#

actually we cant fix that i think

limber skiff
#

the fucking error

austere talon
#

it's a chrome error

#

not a discord error

limber skiff
#

might be able to put an error event handler

#

okay so its here

#

hmm maybe not

#

it does have a onerror

austere talon
#

😭

limber skiff
#

that's so stupid

nimble plaza
#

shitty hacky fix idea:
in electron is it possible to specify default settings for devtools?

#

ie: console filter

limber skiff
#

anyways if we dont have any other ideas I'm merging the plugin soon

austere talon
#

your best bet is a devtools extension

#

what if we made patch helper a devtools tab

nimble plaza
#

scary

dusk blaze
#

independent from vencord

#

would have things such as disable all plugins

austere talon
#

wait we could make a devtools tab for updating vencord

dusk blaze
#

clear themes

austere talon
#

make

dusk blaze
#

okay

austere talon
#

will be officially added if good

dusk blaze
#

okay

#

I will get on that in probably a year

austere talon
#

BOOKMARKS

#

in inbox

#

so good

charred monolithBOT
nimble plaza
#

wait idea

#

what if we patch the spotify ws url to

#

?aaaaaaaaaa<repeat>&access_token=blabla

#

so console shortens the url

austere talon
#

there's no point to

limber skiff
austere talon
#

since we can't hide the spotify token anyway

limber skiff
#

why now that I merged 😭

austere talon
nimble plaza
#

aw

austere talon
charred monolithBOT
#
[Vendicated/Vencord] branch deleted: console-janitor
limber skiff
#

thank you so much nitro

fallen vale
#

💀

woeful sable
jagged cloak
#

you love RPCServer

limber skiff
#

you mean that I should have added that?

jagged cloak
#

i mean sure if u want

limber skiff
#

i added it partially

#

but if there are more annoying loggers tell me

jagged cloak
#

i think its because i have that one dev option on actually

#

i dont remember if rpcserver spams like that without it

#

just kidding it does

#

guess i was hallucinating again skulley

limber skiff
#

which one exactly

#

send screenshot of a log

jagged cloak
#

this stuff

#

its either super noisy or not at all

#

depends

#

i just have it filtered all the time though

dapper crest
jagged cloak
#

-RPCServer

#

literally just that lol

pine wadi
#

esbuild 0.15.18 (released 2022-12-05) doesn't support RegExp literals using the v flag. may i update our esbuild dependency to a newer version?

#

i'll go for submitting a standalone "esbuild": "^0.23.0" then

charred monolithBOT
pine wadi
#

actually, @limber skiff, since you're around, could you review #1661 please?

limber skiff
#

I'm gonna let vee see it, but if they dont I will do

pine wadi
#

it really is just (all in scripts) pnpm run for future-proofing, factoring out the lint scripts in the style of "lint": "pnpm run lint-user --ignore-pattern src/userplugins", adding test-user and testWeb-user which call -user variants when appropriate, and adding .m{t,j}s{,x} to the eslint extensions

#

if you're strongly opposed to linting src/userplugins then you should be able to cherry pick just the last commit just fine

charred monolithBOT
charred monolithBOT
charred monolithBOT
charred monolithBOT
#

Adding this to your quick css makes the roles scrollable if the roles list is longer than the height of discord

[class*="userPopoutInner_"] >
[class*="body_"] {
    overflow-y: scroll;
}

[class*="userPopoutInner_"] >
[class*="body_"]::-webkit-scrollbar {
    display: none;
}

https://github.com/Vendicated/Vencord/assets/71079641/98d0d937-aa46-4911-b63f-0a1a04958901

charred monolithBOT
austere talon
#

dude i fucking hate the web chunk

#

it takes so long to load

fallen vale
#

uwu

glass jasper
#

jumpscare

charred monolithBOT
pine wadi
#

buildWeb --watch seems weird but i think it was that way before?

charred monolithBOT
pine wadi
#

oh, @dire trench isn't here

austere talon
#

hes banned

#

p sure

fallen vale
#

lol

#

imagine getting banned

fallen vale
#

oh, nvm, misread it

pine wadi
charred monolithBOT
#

[!NOTE]
This PR is a patch stack. Do not squash these commits. This patch stack depends on:
+

I wanted to use RegExp literals using the v flag, but esbuild was too old. Whee.

The build scripts are first moved over to TypeScript (.mts) so upgrading with esbuild is sane. VS Code wouldn't show errors for the previous .mjs files for me, at least. tsc will catch future type errors now. (I can split this out to a separate PR if you really want.)

esbuild is then mov...

charred monolithBOT
#

Very funny - 1. why would you have that many roles - 2. I haven’t really tested it with that many roles to be fair and I have large screens - 3. question is how to solve this, scroll(?), limit the amount of roles (?) like 50(?), add a setting to set the amount(?)

i was thinking scroll but like the same way as if you were hovering over a long status bubble within a profile and it just shows the rest of the content. here's an example of how i copied the same behavior but with my about me.
...

charred monolithBOT
limber skiff
austere talon
#

what if we made it so that it splits all modules in dev builds

#

so we don't have to deal with that massive chunk anymore

limber skiff
#

oh don't

#

I've done that for testing in the past and I'm pretty sure it was worse

austere talon
#

why?

limber skiff
#

devtools doesn't like so many source files

austere talon
#

oh

#

i'll test

limber skiff
#

okay

rugged spire
#

as someone who has used that for long enough that I ended up heavily forking it

#

there is (probably) a reason why it hasnt been merged

#

not that it is a bad plugin, it's just

  • very complicated
  • doesnt actually hook around Discord's history wrapper
fossil inlet
fossil inlet
#

might fork it myself and add those features

rugged spire
#

also the fact it feels laggy just because of expecting tabs to be instant

rugged spire
#

isnt there an insane 500 line util file dedicated just to tab management

fossil inlet
#

also this favorites section superseded it for me

rugged spire
fossil inlet
rugged spire
#

i need to Shove CSS into it

rugged spire
#

oh

#

discord is stupid

#

i see

fossil inlet
#

no search

rugged spire
#

respectfully why did you apply a windows icon skin on linux

rugged spire
#

cursed

verbal pumice
shut vineBOT
#

Alright @verbal pumice, in 10 hours: look through this

💡 Did you know you can set your timezone with "!timezone set"?

rugged spire
verbal pumice
#

yeah

#

wasnt going to

#

i just gave up on trying to do anything more w the plugin after horribly failing to add keybinds

charred monolithBOT
#

[!NOTE]
This PR is a patch stack. Do not squash these commits. This patch stack depends on:
+

For ease of authoring patches:

  • PatchHelper's find now canonicalizes RegExps (e.g. /\i\.foo/)
  • PatchHelper's find supports single & double quoted string syntax (while still allowing unquoted strings)
  • PatchHelper's match supports RegExp syntax
  • PatchHelper's match supports single & double quoted string syntax (while still allowing unquoted strings)
  • PatchHelper supports ...
austere talon
#

tf is a patch stack

pine wadi
rugged spire
pine wadi
#

alternatively, "atomic commits"

pine wadi
austere talon
#

what in the world is the point of this

pine wadi
#

because esbuild is something i really didn't want to break, this is meant to make reviewing with the changelog open easy

#

you can see what i thought needed changing for each set of breaking changes on esbuild's side

#

also if something did break that i didn't catch you can have a better experience bisecting

rugged spire
#

git bisect cant fix merging in fucked up code that was force pushed over on the remote

pine wadi
#

?

#

that sounds like a separate issue?

rugged spire
#

it is trolley

pine wadi
#

i wanted to reduce the pain of debugging a broken build if that happened. the commit list is longer but you can see that most of them only change 2 files and skip them with a better amount of confidence in manual review once you've confirmed that the breaking changes on esbuild's side are irrelevant

pine wadi
# charred monolith

also i think pnpm run build from here is now faster with the new logging handling. it at least feels faster than pnpm run build --summary from here, but that may have extra delays to preserve terminal output because esbuild kept on outputting over itself in my local tests (see attached). i haven't benched any of this.

#

esbuild technically does lock some when logging but it's obviously not enough under concurrent builds across multiple contexts.

#

also, current pnpm run build output with an error. a bit more compact but i think it's good enough

charred monolithBOT
limber skiff
#

vee loves sending me merge conflicts

#

I would keep using Vencord.Plugins here

#

it's all executed later so it causes no issues

#

and importing plugins might cause circular

charred monolithBOT
austere talon
#

it cant

#

only if u use on top level

#

theres alr circular dep somewhere

limber skiff
#

how do you know?

rugged spire
#

plugin settings trolley

charred monolithBOT
frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

limber skiff
#

well it works

charred monolithBOT
limber skiff
#

gonna keep that as a draft

charred monolithBOT
#

[Depends on utility functions exposed in #2662]

About Plugin

Discord often hardcodes colors despite having css variables for all it's colors.

For example, --primary-160 is #ebedef, but in the code, they have hardcoded the color hex instead of using the variable:

.defaultLightModeCustomGradient_e77fa3 {
    background: linear-gradient(rgba(0,0,0,0) 20%, #ebedef 100%);
}

This causes issues for theme devs who want to make stuff by directly modifying color variab...

desert cosmos
charred monolithBOT
still wasp
#

Fix 093732189123 issues with themes instantly

long stream
# charred monolith

This is very prevalent when using ClientTheme and looking at "channels and roles"
huh

austere talon
#

that's fucking hilarious did discord fix it at the same time

austere talon
still wasp
#

No way

#

😭 😭 😭

austere talon
#

inb4 they unhardcoded every colour

long stream
#

was it ever broken?

austere talon
#

this is on canary btw

dapper crest
#

are you on canary or smth?

austere talon
#

yesterday

still wasp
#

Discord is just tactically planning updates to piss off vencord contributors

dapper crest
long stream
long stream
dapper crest
austere talon
#

why don't you?

still wasp
#

Vee is a sacrifice to the canary gods

dapper crest
austere talon
#

do u not have vesktop

austere talon
#

missing out

dapper crest
#

so true

#

I should use vesktop

#

how do you point vesktop to a custom vencord install?

woeful sable
#

in the settings

still wasp
#

Smh

dapper crest
still wasp
#

Shrimple

long stream
#

whyyy

#

is my dev install that broken

still wasp
# long stream whyyy

I'm no expert but i'd say it cannot read properties of undefined (reading 'Settings')

austere talon
#

disable BetterNotes

long stream
#

it errors regardless of if its enabled or not

dapper crest
#

oh those border colors probably won't work

#

need to fix how I insert var(...)

charred monolithBOT
dusk blaze
#

we need to disable issues on vendicated/vencord and make some private issue tracker exclusively for contributors
soo many people ignore requirement

frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

ShowAllRoles (had no effect):
ID: 579285
Match: ```
/return null!=\i(?=?\i.slice)/

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

still wasp
#

Ruh roh raggy

austere talon
charred monolithBOT
rugged spire
charred monolithBOT
desert cosmos
# charred monolith

some people manage to turn off simplified profiles
sounds like their issue in my opinion

charred monolithBOT
#

btw I'm not sure to where place voice status in user popup. originally this component located between send message section and roles section. in simplified profile already exists some "voice status"(activity), I just override it by section from this plugin and now this one located ABOVE roles section. would it be better to move it as in original plugin (under roles)?

charred monolithBOT
desert cosmos
fallen vale
shut vineBOT
#

@verbal pumice, <t:1720032162:R>: look through this

weary bolt
#

!remindme 8h wake the hell up before i slap the living shit out of you

shut vineBOT
#

Alright @weary bolt, in 8 hours: wake the hell up before i slap the living shit out of you

charred monolithBOT
fossil inlet
#

but it should be at least a toggle

rugged spire
glass jasper
charred monolithBOT
shut vineBOT
#

@weary bolt, <t:1720070107:R>: wake the hell up before i slap the living shit out of you

charred monolithBOT
charred monolithBOT
weary cave
#

@austere talon I think you are better equipped to answer this than me, I don't know what the point is I just re-implemented it for the latest https://github.com/msfragala/eslint-plugin-path-alias/issues/1#issuecomment-2209332213

GitHub

I'd like to request you disable minification on the TS build, as this makes patching the package difficult. I don't think there is much to be gained by minifying the package, considering th...

charred monolithBOT
fallen vale
charred monolithBOT
fallen vale
#

someone's obsessed

fossil inlet
fallen vale
charred monolithBOT
fossil inlet
frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

ShowAllRoles (had no effect):
ID: 579285
Match: ```
/return null!=\i(?=?\i.slice)/

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

kindred estuary
#

pain

austere talon
kindred estuary
#

i’m aware

limber skiff
#

I guess it's fine to keep it working for now

charred monolithBOT
charred monolithBOT
#

omg epic sigma skibidi rizzler code

that’s cringe, not funny

damn, i cant even make a brainrot joke without some weird gay mf getting mad lol

you’re literally a full stack developer making dumb jokes like this, and you run your own website, get a life instead of being so brain dead

fallen vale
charred monolithBOT
#

omg epic sigma skibidi rizzler code

that’s cringe, not funny

damn, i cant even make a brainrot joke without some weird gay mf getting mad lol

you’re literally a full stack developer making dumb jokes like this, and you run your own website, get a life gen ass

peter-griffin-popular-vibe

austere talon
#

dude why is this even here

#

this is some fork commit 😭

woeful sable
#

why doesnt github mobile have the ability to read commit comments

#

😭

charred monolithBOT
long stream
#

for some reason those commits count

charred monolithBOT
charred monolithBOT
glass jasper
still wasp
#

😭

frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

rugged spire
#

hop online first thing i see is 1 minute ago deleted message of someone being a clown husk

limber skiff
#

@austere talon can you make a repo for the code you already have for the new reporter?

austere talon
#

sure tomorrow

#

but i didn't work on it any more

limber skiff
#

no problem

#

I just want a repo to work at and the code you already have for a start

austere talon
#

i mean there's not much more to do, i might just finish it tomorrow

#

also it's in go not js

limber skiff
#

I've done stuff with go in the past already

charred monolithBOT
charred monolithBOT
limber skiff
fallen vale
#

yes ;-;

limber skiff
#

if it's there it's for a reason

#

my god

#

stop making useless reviews

fallen vale
limber skiff
#

okay sure if it was another contributor or first time contributor

#

but it's vee, he wouldn't do stuff for nothing

#

it's not the first time you are making useless comments/reviews

fallen vale
fallen vale
#

and then i stopped doin that, and then the checkbox was removed shortly after

limber skiff
#

the reason is in one of the chats, I think plugin dev

#

it does not matter however

fallen vale
#

whats the point of opening a pr, if a review asking a genuine question is going to be met with "really, my god, stop making useless reviews"?/genq

limber skiff
#

my god

fallen vale
#

im not tryna be rude, just understand the situation a bit better

limber skiff
#

it's literally the owner of the project

fallen vale
#

i get that

limber skiff
#

and thats enough of a reason to not ask why that was placed in that PR

#

it's needed changes which are gonna go to dev regardless

#

vee just took the opportunity to do inside that pull requests

#

my pull requests do that all the time

fallen vale
#

okay, thank you/gen

fallen vale
limber skiff
#

not even my project

fallen vale
#

no ik

limber skiff
#

that was just not needed at all

#

the other one is valid, a good suggestion

fallen vale
#

i was sayin that if thats what the owner of the project thinks, and thats whats agreed on here, its non beneficial for anyone to argue against it

#

alr, i hid the comments

charred monolithBOT
fallen vale
#

i did mean to response
i need to go to sleep

charred monolithBOT
limber skiff
#

reporter tests how long a patch took

charred monolithBOT