#๐พ-core-development
1 messages ยท Page 244 of 1
yea thats what im doing
im not redoing all those files 
@limber skiff uhhh
no patches fail, but is this normal?
LMAOOO
boolean cast, idk
you ran it too early I think
includes returns boolean
to have things more clear
omggg
@fossil inlet makeLazy is just fucking broken
it doesnt work with false...
export function makeLazy<T>(factory: () => T, attempts = 5): () => T {
let tries = 0;
let cache: T;
return () => {
if (cache === undefined && attempts > tries++) {
cache = factory();
if (cache === undefined && attempts === tries)
console.error("Lazy factory failed:", factory);
}
return cache;
};
}
change it in your PR too
incredible
@cobalt atlasies wont makelazy just not work with booleans
because it just checks that cache is truthy
whoops, sorry random
I sent you the fix lol
guhhh i love discord being slow to load messages
90k members ๐
cause it wasnt meant to be used like that ๐ญ
since when is it meant to be used in a specific way
@fossil inlet kill those !!
also make it return false here
seems super overengineered and unnecessary
just have two patches and add a comment
just keep itt
no i dont like it
this is gonna be shit
#1337479880849362994 is going to be spammed
just like it already is
and adding noWarn is even worse
just add a todo or fixme
log being spammed would be 
but having the log spammed is just horrible
it's fine
you cant even track properly if something new breaks
log being spammed is not fine
?remind 2 weeks check if discord has rolled back
Alright @fossil inlet, in 2 weeks: check if discord has rolled back
that skip thing is horrible
better than logs being spammed
what else are you supposed to do
you cant track if discord reverted or not without something like this
and this change is so dumb they are likely gonna revert
but right now a lot of things are broken and we gotta fix
I already added the build number stuff before to be able to push early fixes
but this one you just cant track with a build number, since it's a bundler change
- without something like this people in support might get the wrong idea when asked to ss their console
just overall more confusing
it really doesn't matter that much
just fix it and if they revert it we revert the commit
you're overcomplicating and overengineering this a lot
we can notice and revert so quickly no one will even notice
the logs will still be spammed, that matters
there's no downside in doing this besides duplicate patches
which you already told us to do anyways
don't then
if they revert, we see it in reporter, revert the commit and done
its not a big deal
if they do revert, we wont have 5000 people asking for help in support
I dont get why you are so against it
it's not a one time thing, we can use this in the future too
of course we could just push things quickly if they change something, but extension exists
@fossil inlet holy shit the bundler cooked so hard here
what even is this
used to be an object spread
they just inlined the whole thing lmao
it was this
return (
e.forEach((e) => {
0 !== e.settings.length &&
(!0 === e.divider && i.push({ section: l.ID.DIVIDER }),
Vencord.Plugins.plugins["Settings"].addSettings(
i,
e,
l.ID,
),
null != e.header &&
i.push({ section: l.ID.HEADER, label: e.header }),
e.settings.forEach((e) => {
i.push({
...t[e],
tabPredicate: () =>
null == n ||
(e === M.s6.SEARCH_NO_RESULTS &&
0 === n.size) ||
r.has(e) ||
n.has(e),
});
}),
e.settings?.[0] === "CHANGELOG" &&
i.push({
section: "StartupTimings",
label: "Startup Timings",
element:
Vencord.Plugins.plugins["StartupTimings"]
.StartupTimingPage,
}));
}),
i
);
you should message brandon and tell him the bundler is cooked
who 
๐๐ป
telling him to fix it and also enable the merge modules option
surely their build reduces so much like that
i fear that would cook 200000 patches
yeah
why do u want merge modules ๐ญ
itd be bad for us
only slightly better for finding stuff from small generic modules
but worse for everything else
merge modules is cursed with rspack (from my testing)
@fossil inlet, <t:1739929078:R>: fix
i feel like that patch needs to be rewritten
it doesnt feel right
I know it's ugly
but it was on purpose lol
they kept changing a lot so I made it as generic as possible
honestly just don't have that patch
either add it to the vencord category or don't add a page at all, have it render in the plugin settings
it's honestly pretty weird that it's down there
yea, it could prob just be added to vencord settings
rendering in plugin settings is husk
(tbh i dont think anyone even uses it)
I inverted shouldSkip to shouldApply
alr
already pushed to your pr lol
no
love
@limber skiff shouldApply doesnt make much sense if you want this to be used in the future as well
why?
wait, im stupid
for some reason, i thought you were renaming spreadEnabled
^
there is, but im pretty sure emotecloner makes no use of type 1 stickers thus no reason to add it
@dusk blaze 
Not Fixed by this pr yet
- SHC hidden channels are showing up as unread all the time
ill fix it tomorrow morning if someone hasnt done it already
There's some jank in it (it makes youtube embeds really small) but I'll upload it later
I had a quick look into it and the channel prop being passed to isHiddenChannel is undefined for some reason, I don't have time to go further into it though (I'm slacking off work)
i just updated my vencord fork with the changes of 1.11.4 and .5 and it seems like platform indicators css is weird
the phone icon is higher and the 2 platforms are aligned on the bottom instead of the top and pushed to the right, was that intentional ?
also the order changed and is inconsitent
Wait, how did you notice such an insignificant detail?
idk i'm built like that
it's called the combo of ADHD and Autism traits, details stand out to me
Nuckyz did some work to make it more centered in the div overall a few weeks ago
also the order being inconsistent is this
like sometimes one is before the other and it's weird
probably just the order discord reports them in
i'm gonna be the annoying person here, but i like it like it was before
https://xkcd.com/1172
sadly true
^
people do insane shit and complain when their bug abuse doesn't work anymore

happens all the time
it's consistent even with different logging order
it seems like the classic nothing is in a defined order because map or set or smth
ok but that didn't include phone
it's the phone that stand out to me a lot
phone be too high just like me after i took... nvm forget that /j
I think they're more aligned to the centre rather than top now
i mean technically it cannot be centered as one is 1px bigger than the other
it's just offset in the other direction
ok i found the issue
it's not aligned with what discord do
discord stuff is aligned higher
bruh 
I'm never gonna join anyone's clan, I don't need to see them 
this will drastically affect fishing season
well the clan experiment ending anyway
They gonna keep the tag for everyone smh
the phone and web should be 1px higher i think
I wrote a snippet
You can go find it it still works
I've been hiding them ever since they came out
wtfff
vext
The Vencord Chrome Extension is up to date! (v1.11.5)
oh cool
Isnt there a setting in accessibility to hide them?
So there is
StatsFMRPC is a plugin to display what you are playing on stats.fm. (Based off of Lastfm)
Why would you use this over spotify status? I don't think this is very useful. If anything it should just be integrated into the last.fm plugin
Thanks for your contribution nevertheless!
that means vendicated is actually reviewing plugins
i hope he won't close my pull request 
It'd probably already be closed if it were a quick easy rejection like that one
true
then i have my chances
FontLoader
A plugin allowing you to search and use any font from google fonts directly from your discord allowing you to preview it in the app.
that would be useful 
Can be done with
#๐จ-css-snippets message
Patch by BetterBanReasons had no effect (Module id is 98746): /[({((?:name|value):\i.intl.string(\i.\i.[A-Za-z0-9]+),?){2}},?){3}]/
Doesn't work anymore.
asdasd
3
3
1
asd
1026532993923293184
husk
it's a problem with the phone icon, which has a different height
also about this
The phone is nearly normal sized, in comparison, console is wild
I know they are not centered in the container
they are placed a bit lower, to match more with the date and nickname text
discord elements are inconsistent too, so I chose the one which looked better, which is aligning a bit more down
Personal opinion but i preferred when they matched with tags and name
Because those are tied to the users and not the date imo
Also the fridge (i really thought that before i realized it was a console) is really big, idk if it could be slightly smaller
I had some screenshot hold on
.
huh
why are they being inconsistent
now sometimes it's aligning in one way, other times in another
2 looks so weird with the speaker being way too low
It's like if it had gravity
While 1 it feel floaty
1 looks weird comparing the speaker with the text in the right
also this is just different lmao?
Eeeeh
Thing is my brain consider the team of the right to be part of a different group
same css, yet different
Icons are all a single groups with tags and role icons imo
Idk why but i feel like cursed discord CSS is acting again
Ok what
they are placed super low too
๐
I wonder if at that point i shouldn't just patch the entirety of discord CSS
nah do you wanna know the problem?
They CSS is some spaghetti layering
:pain:
and they manually apply top or bottom
margin-top or magin-bottom
to center each one manually
Do UI designers really not care about it ?
okay so, 21px
lets see the other
21 too
but you can clearly see the difference on alignment?
And the one different thing is the date after
I wonder if old messages have different CSS
Is it some inherit bs ?
Like somehow it inherit diff CSS or like it gets calculated differently because funni layered CSS
Opening discord CSS be like :
https://tenor.com/view/explosion-on-the-face-jokey-smurf-smurfs-gift-exploded-kaboom-gif-24671707
oh uhmm
my vencord isn't working properly
on canary
ah I broke something
okay
anyways
for that platform indicator problem you are right
phone is smaller in height
so it's alignment gets scuffed too
And fridge is bigger, but it gets properly centered cause 2px smaller than the container
the fridge ๐ญ
I'll look into this another time
specially this weird moving css that is inconsistent
I just had it too lmao
Also idk if it could be done easily, i have thought doing it with index CSS or smth, but make the icons always in the same order
Like if it was always desktop, web, phone, fridge, it'd be nice, but just a consistency nit
i havent pulled yet
yea
yeah it's possible with the JS part of the plugin

GUHHHHH WHY DOES MY DISCORD KEEP BREAKING ON RANDOM REPLACMENTS IN CONSOME
@austere talon we switched to noWarn, is it good now? https://github.com/Vendicated/Vencord/pull/3237/files
salad
salad
I think it's more that the doc is on the wrong line
@limber skiff is there anything else blocking the fixes?
nop
10k stars ๐
this might actually be a sane idea
reporter will update a #1257025907625951423 post
The following plugins might be broken: with a list of pluggers
if it were added, there should be venbot commands to add/remove plugins from that list
eg: shc is broken rn, but no patches are broken
3e524f9 WebpackPatcher: Improve getBuildNumber; Allow r... - Nuckyz
b5ef858 Fix plugins broken by latest Discord update (#3... - sadan4
vtest
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
@fossil inlet god damn you were faster than me lmao

@grizzled halo can you update the title in known issues
good timing cause I just woke up again
@limber skiff are you going to tag for browser given we have patches with nowarn if they revert
wait a bit
alr
Patch by BetterBanReasons had no effect (Module id is 98746): /[({((?:name|value):\i.intl.string(\i.\i.[A-Za-z0-9]+),?){2}},?){3}]/
Doesn't work anymore.
Resolved
they're out of date dummy
Megred Statsfm into lastfm, Added Devs of Statsfm into Contributors
Originally from https://github.com/Vendicated/Vencord/pull/3242
Hidden channels were not displaying as when using ShowMode.HiddenIconWithMutedStyle
I just stole sadan's fix for the unread patch 
Changed the default "Mute Message (only self for now)" config value to lowercase to keep consistency with other default values like Unmute Message and Deafen Message
PR made from what you requested
(https://github.com/Vendicated/Vencord/pull/3242#issuecomment-2671764425)
You've skipped the main part
Why would you use this over spotify status? I don't think this is very useful.
PR made from what you requested
(#3242 (comment))You've skipped the main part
Why would you use this over spotify status? I don't think this is very useful.
LastFM and StatsFM both provide support for Spotify. The only difference between them is that LastFM is has an api which allows any app to integrate it, while StatsFM has integrations for only Spotify and apple music
would this be worth implementing to remove one less hacky replace
because its kinda redundant with this
Damn, time to close this then 
he didnt add himself to contribs?
nop
just changed the casing of one thing lol
me when i pronounce a capital M instead of lowercase
I didnt add myself to contributors when I made my pr for changing some icons
didn't feel it was significant enough
yop
ive never had a pr merged
(the merged button is added by RGH, so it doesnt have a counter)
what is rgh
you will use
nop use new github ui
booo, this isnt showing all languages tho, i have typescript, yaml and xml in this repo too
one of my repos
nix
why does it look like that lol
its wild that rust is my 2nd most used language, when i wrote a SINGLE hello world project in rust, and it generated so much loc that its my 2nd biggest lang
๐
how do i get that 
this is my craziest project 
that smells like a uni project form a mile away
that smells like a pride flag from a mile away โโโโโโโโโ
ah right i forgot
vtest dev
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
Sick
pride flag not trans ๐ญ
sorry, there's too many flags to remember โโโโโโโโโโโโโโ
Submit and find out
review what
extension
heh.
imo, this should be something like imgur incase github yeets the image.
insane
Why would github yeet that image
What if imgur yeets the image
what ๐ญ
what ๐ญ
what the banana is the point of them messaging you on discord
@glass jasper holy shit stop nerd reacting every little thing you see its annoying
vee plus brainrot
vtest dev2
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
vtest
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 

youre all good
Content
- Automatically Select Roles of User has using that option if you have server permissions
- I alrady know the plugin requests are closes but this one is really productive for server builders & also for modrators to view as users perspective of server much faster.
Request Agreement
- [x] I have read the requirements for opening an issue above
I feel like this would be better as a part of betterRoleContext
What
Adds a new option to LastFMRichPresence plugin
"Always Hide Art"
- "Disable downloading album art"
This is a boolean which is checked at getLargeImage() and skips returning the track's imageUrl, instead deferring to either the placeholder or null (lastfm logo) depending on other options.
In essence, it acts as if all album art is impossible to find
Why
Last.fm art is uncensored. I would personally not like to flashbang whoever clicks on my profile with Death...
statsfm is lastfm but bad...
Discord patched error:
a similar problem occurs (below is an error with the Cli, but the same error occurs in the UI version). If I do install it, an error occurs when opening Discord. When you uninstall, the discord opens quietly. I tried everything, installing certificates from Google, using a proxy connection, and modifying DPI packages. Nothing helped.
ERROR Failed to send Request Get "https://api.github.com/repos/Vendicated/Vencord/releases/latest": tls: failed to verify certificate: x509: certificate is val...
husk
use our support channel
i want to make some changes to code in src/ but i have no way to test anything atm
i could setup an ide on my arm chromebook but i think it would be too much hassle. do i just add my code and make a pr?
im planning on adding a setting to the showHiddenChannels plugin
fucking support banned without talking to me normally. Why can't it be solved here?
Because you was selfapping some signature/ advert
you can get a github workspace
and code from there
just press , while in the repo
true
ive had bad luck with it in the past with our crappy internet but i guess i could try it again
also do i use the dev branch first or does it matter
we don't allow selfbots, especially those adding ads for your software. that's why you were banned
idiot
should add the json clause to the license
you're not allowed to use vencord for evil
is this serious
no lol
good

because then vencord would become source available...
no no it would just be not free
should i?
i dont wanna go making a pr to main only to find out that i should have made a pr to dev
It's not in the license, but we do have a couple of well trained ninjas.
support ninjas
we don't allow selfbots, especially those adding ads for your software. that's why you were banned
I didn't even know that, you didn't give me a chance to explain myself or ask any questions. I could just turn off my damn watermark, lol
your pr will be merged to dev anyway
issue tracker ban him
me when i have to wait 10 minutes and go through rules screening to get in the server
Me when I deliberately add weirdass obnoxious functionality and get banned for no reason
You was asked, but didnt
wtf
I just logged into your server. Yes, it's my fault that I didn't check the rules, but no one even explained anything to me properly, and they immediately started banning me. I was given a warning about the watermark, I explained that the bot is personally mine and not public, in fact it is not an advertisement, and if in your opinion it is, then what good is it to me? I did this because I can and want to + I wanted to test the message processing speed of my program (it's not very fast)
LMAO
who the actual fuck adds a watermark to their NORMAL messages
i can see like special things maybe
but that shit just annoying wtf
oh ok
should i go to #๐งฉ-plugin-development since im trying to add a feature to a plugin
ok so since i have no idea what im doing im going to make a pr that has the setting in the showHiddenChannels plugin, but it does nothing yet
Add useless hideHiddenChannelsAndShowInMessages setting in the showHiddenChannels plugin.
Hopefully this acts like a feature request to the plugin and gets implemented. The reason as to why this is incomplete is because I don't know what I'm doing, and I only copied and pasted a bit of code.
if its plugin related ask there, otherwise it will prob get burried here
ok
set it as draft
done
ty for telling me, i didnt really know you could do that
(ive seen the button but its never crossed my mind)
ya np
why open it prematurely are you dumb
you know where the channel is to get help implementing it
๐ญ
i dont know javascript
i want to learn but im too lazy
i guess i can give it a try
good idea but just do it bruh ๐ญ
I don't understand the point of the setting
I guess if you just want channel mentions to work but not have them in the list?
This is a completely reasonable thing to want
Yeah I just didn't really understand what they were going for until I was speaking it out
yeah lol
What happens when the bug or crash occurs?
When viewing a Hidden voice / text channel, There is an error stating "Invalid time value".
( screenshots added for more context )


What is the expected behaviour?
I expect it to show the permissions preview
How do you recreate this bug or crash?
-
Open a hidden text channel
-
Open a hidden voice channel
Errors
`...
use our support channel for support
Hello! ๐ I made a small pull request 6 months ago. Is there anything i can do in order to have it reviewed and merged, or should i just wait indefinitely? Thanks in advance! ๐
try mentioning and even linking your PR
sorry i was just about to do that, my bad
stalk this channel and wait for nuckzys to be merging/doing work then ambush him
so i should just wait for him to be around here? i see, thanks!
should probably just use a proper file input instead of reinventing the wheel with a button
what? the button was already there, i just added the drop event to it, thats the entire thing xD
Ping vee until they merge

crazy strategy
or vee
at some point one of them go on a merge session and you sneakily mention your pr
sometimes you don't even have to wait you can just mention it infront and depending on how they feel they'll do it
I've done it successfully before ๐ญ
I love when discord makes a change that needs me to change one character in my replace regex

vban
Failed to ban @charred monolith: DiscordRESTError: Unknown User on PUT /api/v10/guilds/1015060230222131221/bans/1015063255854633081
gonna bump u down a cliff soon @odd heath
He said the thing
By husking you agree to review within 2 business days in Germany
The last time I looked at the code for this PR it gave me a headache.
Could you perchance add an update to remove blocked users from voice chats as well? Love this plugin btw.
i was trying to do something in rspack recently, and it turns out that they have so many webpack plugin hooks implimented, but only in rust and not exposed via javascript
Isnt "Bytedance" the tiktok people?
yes
iirc webpack was created for tiktok
Insane if true
Kinda like airbnb being the ones to create lottie
What?
Just not the kind of company I'd expect a new animation format to come from
Insane
Render After Effects animations natively on Web, Android and iOS, and React Native. http://airbnb.io/lottie/ - airbnb/lottie-web
Not sure if it's a discord thing or a vencord thing but going to the intl tab in Discord's dev tools crashes Discord
And then because they make it reopen dev tools when you restart it just keeps crashing until you clear the last tab in local storage
That tab isn't super important but considering you guys were messing with proxies and intl stuff it might be worth flagging in case it is vencord related
I think they broke their own tools lol
prob discord issue

consequence of disabling spread syntax in the bundler causing this broken code?
I imagine it's more likely the change to the intl proxy
nop
guh
doesnt makse sense i dont think
prob the proxy

this description seems a bit redundant
It is but a description is required, I'm not sure what else to put there. Maybe a link to DeepL?
a link to the token page perhaps
Is there any interest in standardising the style of setting titles like this?
If so I may as well throw it in this PR too
Maybe?
If you do
DO NOT use it as an excuse to skip out on not fixing the casing
Visual Refresh does not do a text transform on those elements
I've already fixed the casing regardless of style so we should be fine
Good to know though
As part of just generally tidying up the look of settings I've also changed switch option titles to match every other type.
I actually prefer the non-caps look tbf but it's probably better to bring switches in line than the other way round
some feedback on the patch would be nice
fixes #3136
If they're that far apart could they just be split into two matches?
I haven't looked at the module
not that far, just check out the module
A similar plugin (#3138) that used AI has already been denied, using AI for stuff like this is a bad idea and once again yet another thing that requires an API key.
Its just not built for something like this and prompt injection is very much possible to do even accidentally, its just not a good idea, use a proper engine built for translation.
Modern translators also use ai, just a different architecture
yeah but they're actually built for that
A similar plugin (#3138) that used AI has already been denied, using AI for stuff like this is a bad idea and once again yet another thing that requires an API key.
Its just not built for something like this and prompt injection is very much possible to do even accidentally, its just not a good idea, use a proper engine built for translation.
I totally get your concerns about AI and prompt injection risks, like you mentioned with the previous plugin (#3138).
I opened this PR beca...
Should use the FileInput component, many plugins use it so you can use as a reference
Closing as this fix does not actually change anything. If the length is 0 the iterator will also be empty and none of the loop code will run.
You should just export another method from the plugin to transition to the channel using the ChannelRouter, instead of copying the onClick from the other component, that should make the patch way more simple
@limber skiff im really confused about something
so i was looking at how the context menu was found
in the module where the names are mapped, 830447, name is defined on a bunch of different exports from module 828214, however when i took a look at that module, all the exports were just null
how does this work if the exports are all null
because each function is a different reference
they arent actually called
they map references -> components
it's absolutely shit
patched by ""?
havent gotten around to fixing it yet 
been working on other things
is it my fault or yours
mine
(i think)
if not ill fix it or let you know
its not like a bunch of people are using this anyway
the exports are not null
the exports are function() { return null }
Should use the FileInput component, many plugins use it so you can use as a reference
I've only found one reference to "FileInput" in src\components\VencordSettings. Is that the FileInput component I should be using? Thanks for the review!
im confused now, because they're called as JSX components all over the place
React.createComponent(x, {...props}) doesn't actually call x
do they map over the children and replace them in some other funciton
guh, i understand it now but it just seems a lot more confusing that it needs to be
the dev who made this was smoking
oh yeah and vencord needs a patch for this lol
or else they were impossible to map properly when webpack finding
yea, i found that then got lost 
that's done by MenuItemDemanglerAPI
yeah idk why they did it like that lol
seems pointless
Yeah that's the one, however I'm not sure if it actually supports dropping
they just implemented getOwnPropertyDescriptor wrong for the intl proxy
I'm gonna pr a fix for it
here we go again
they added *110* new languages to google translate. we got shit like Dinka. Afar. Chechen. Udmurt. this changes the game
time to update the plugin
you husk every other dm I send you
why is it hardcoded 
why
there's a search bar
because if you wanna dynamically fetch it you have to use the api which requires an api key
@fossil inlet I found a way to generate discord message ast
it's just a format called ICU https://formatjs.github.io/docs/icu-messageformat-parser/
Parses ICU Message strings into an AST via JavaScript.
sick
now how can we integrate this with discord-intl
I wonder if we could parse at build time
to avoid bundling formatjs
that doesnt sound too hard
tbh
because I suppose it's kinda like a intermediate format
and discord-intl has methods to format it with values
depends what format you want the keys at build time
are you leaning more towards json or js/ts
json in js
we could just json.parse
just insert the string
I think that's how discord does it
just have to find out how to generate those getters they have
i dont think you need to do that
or create our own proxy instance
which is what generate the getters
gotta see what values you pass to it
@limber skiff i think you're overcomplicating it
how so?
what's the use case
you just need a function that returns the AST
why would we need any of this
then you can pass that to the intl format functions
hmmm
since we dont have translations, mainly to use plural stuff
pretty pointless
without having to do all the discord-intl setup
tbh the only use case i can think for this is when we need to replace discords keys with our own
just do plurals manually or use the pluralise function for simple plural
and iirc theres only one need for that rn
this isn't a real need
you only need a proper solution when you want to support multiple languages
but we don't
horror
W
i feel like it doesnt make sense to have other languages when #๐ฅ-vencord-support-๐ฅ is only in english
me when #๐-translation
me when No Access
the patch should be considerably more sane now
(tbh idk why I didn't think of that in the first place)
Provided link did not match any services.
open the intl tab in devtools and find out xd
OHHH, i thought that was inspect element ๐ญ
forgot discord had their own devtools
so silly
yop
Did discord break their own tools?
i think so yea
yes
iirc nuckyz said it's a result of the proxy changes
trolley
lmao
Was a tag ever pushed for the spread changes
nah
Let me know what's next for this then
badges get turned into orbs in latest canary
it's a virtual currency discord is adding
Gross
wdym, in what way
quest badges I think

doesnt look like it will be that hard to fix
the onclick actions and the tooltips both work
discord gets more horror every day that passes
omgg
it's just a silly error
src: e.vencordImage ?? isOrb ? orbImage : getIcon(e.icon),
spot the problem
Does nullish coalescing really have a higher precedence than ternary, iirc nullish coalescing had one of the lowest
it's a logical operator like || so yeah it does
I got that part (otherwise it would be working) but I could have sword ternary was higher than nullish
oh yeah
vtest ev
Now testing! 
Now testing! 
no one apparently
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
@limber skiff i somehow made a plugin that shows this screen ~1s after going to any guild channel, you have to go back to @me and wait a few seconds for it to disappear
how did i manage this ๐ญ
no errors or anything
HUSK
yes as nothing happens if i dont navigate to a guild channel
(and my plugin is patching the member list)
i still havent figured it out 
but i found it only happens when the window is focused
nvm, i think the gateway is prob erroring
You can turn on gateway logging in the dev settings if that's any help
it would have been really easy if i knew it was a react error
I found an issue building when using pnpm 10.5.0
for some reason Vencord has a requirement in package.json for a very specific pnpm version and I'm wondering if I should open a PR to remove that requirement
this happens when building with 10.5.0: ERRORโ Failed to switch pnpm to v10.4.1. Looks like pnpm CLI is missing at ".."
- "packageManager": "pnpm@10.4.1",
+ "packageManager": "pnpm",
^ (this solves it in my case but I'm not sure if there's a reason for it, and I've noticed there were previous "update pnpm" PRs that were closed as unresolved)
use corepack
isnt corepack alr deprecated?
no? corepack isnt deprecated
i cant find anything on the corepack docs that mentions its depreaciton
ah, corepack will no longer be part of node and not supported by node, just its own thing
because nodejs memebers said no to corepack realising its a bad idea, but people like it, so they will make its own standalone thing
minor config change

adding this feature because somehow a react error was logged by one of the loggers, and didn't show up anywhere else in the console
Badge of shame
Quest badge already is

still dont know know how a react error was caught by discords loggers and never made its way to the console
@fossil inlet if I understood your description right then by default it allows everything
no
because error is below everything, which means everything is above it and is allowed
by default it only whitelists error
guhhh i didnt mean the description visually ๐ญ
didnt think of that 
idk 
wait i remember, i was thinking about severity
eg error > warn > log > debug
just switch the order of the select
yop
nop
we never used it before
that could be a good first use tho
better than a multiple option select
yea, its on my list of things to demangle ๐ญ
do you have a find for it?
not yet, but it should prob be pretty easy
we dont need to make a setting component implementation for it
just do a custom for console shotcurts, and make the actual setting an object
with each value as a boolean
i went into the design components and made a list of components that i need finds for for my demangler plugin 
HOW

No
i recall using a find like that blobcatcozy
yop
something like it
that actually looks a lot better then what i had in my head 
really happy with how this plugin is coming along so far 
(i wish react devtools wasnt that jumpy)
yop
I pushed it to your branch
love
gotta fix the checkbox texts which have extra width for some reason
will do that tomorrow once I can
This makes it wildly easier to find what you are looking for in react devtools
Before
After
*not all of those names are provided by this patch, only the ones used in vencord are
๐ญ
its fine, im going to test it all sometime tomorrow anyway
How cursed would it be to identify module exports based on order rather than finds?

This is pretty sick
If even just to help keep your place in the shitty node tree
I want a "collapse all but this node" in react dev tools
I've never touched react devtools tbh
They're good except for the fact they constantly jump around every time anything happens onscreen
With things like typing indicators and member list changes it can be annoying
lmao
and/or a thing like task manager where you can hold ctrl to pause list updates
vp f8
Unfortunately that stops react dev tools from working
Little bit
I installed a more up to date version of dev tools a bit ago but it was kinda buggy and there weren't really any meaningful improvements so I just reverted back
I kinda found a way to do this. If you turn off Expand component tree by default and then select the element via chrome dev tools, then go back to the react dev tools tab it'll have that node expanded with everything else still collapsed
If you select the element via the react one it expands everything you mouse over on the way there smh
Yop
No
No as in "not cursed" or "you will explod"?
I just tried it again and realised the biggest issue (besides the node tree refusing to render sometimes) is that it can't Go To Definition where the source is a patched module (nvm I fixed it)

I feel like this would be the most useful to people who are new to working with discords code, as they wouldn't feel completely lost with all the minified names
this api is really weird. especially the option for findComponentLazy is misleading
bad
I'm not sure how realiable is that
@limber skiff i love typescript https://github.com/sadan4/Vencord/blob/feat/consoleJanitor/levels/src/plugins/consoleJanitor/index.tsx#L109
index.tsx: Line 109
} satisfies AllowLevels as AllowLevels,
lol
tbh idk why i have to do that, because iirc you need to do a const assertion to take the narrowest type by default
so it should just work with satisfies, but it doesnt
nevermind, satisfies asserts const
but then how do you ensure a type inline, without casting it or asserting as const
@limber skiff where did the logDangerously and verboseDangerously methods come from, i cant find them on the mdn page for console
https://github.com/sadan4/Vencord/blob/feat/consoleJanitor/levels/src/plugins/consoleJanitor/index.tsx#L15-L26
index.tsx: Lines 15-26
const NoopLogger = {
logDangerously: Noop,
log: Noop,
verboseDangerously: Noop,
verbose: Noop,
info: Noop,
warn: Noop,
error: Noop,
trace: Noop,
time: Noop,
fileOnly: Noop
};
They're added by discord
ah, they're used by native
We use react types of a higher version than Discord and FunctionComponent can return Promise
you dont

even if FunctionComponent can return a promise, it isnt be handled
But why are you blocked
idk
it's been a few weeks
uhmmmm

๐ญ
can someone review my pr
i don't know the cutoff date on when i can ask or not
it doesn't matter
that's future proofing
@fossil inlet I fixed it
horror
no other way lol
@limber skiff how cursed would it be to patch react internals
why tho
I keep getting insane ideas that I just want to make 
Ill send it if it works
alright
I think I've been staring at the react source for too long 
Worked fine for me
But cursed? Absolutely
The break has been pushed to stable
hahaha gl
did u try to access /.git or something
the only part i use is the build logger
and i have partially done that already
i just need to make a web ui (i suck at UI/UX)
nah, just typed nelly.tools into the search bar
interestingly visiting that gives me a 429 error code instead of a 403/404
ur gonna get banned silly
insane
bruh @limber skiff
what happened
lmaoooo
do you wanna push anything along with that
I just wanna push the fix for now
vtest dev
Now testing! 
FakeNitro (took 8.1ms):
ID: 937889
Match: ```
/1!==(\i).length||1!==\i.length/
Seems like everything is working fine (for now) 




