#๐พ-core-development
1 messages ยท Page 208 of 1
aight
let me test it again
code looks good
you can merge if it's all good
actuall wait
nvm
we did merge it into volumebooster thats good
or wait no
huh
i was proposing to extract it into WebVolume or smth like that which is enabled by default to match expected behaviour
nvm it's not good yet
sound is way to loud on lower porcentages
it doesnt really happen and the audio looks way lower on like 10% than with the plugin enabled
it's so much louder that you can easily see where the volume turns to 0% when you are in like 8%
it's impossible to get a good enough low volume with the booster lol
:( โโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
@limber skiff ill take a look at it and see what i can do
should i release then?
yeah
lets run test one last time
(im paranoid)
oh yeah i still wanna continue working on auto reporter
whats autoreporter?
autoreports
is that just reporter whenever discord pushed a new commit
None
None
None
None
ye
None
None
None
None
I really don't wanna annoy with it but is it possible for my big pr to have a chance to get merged soon
it's been there for so long at this point I just keep doing nitpicks with myself lmao
051bce8 feat(Webpack): Add $$vencordPatchedSource to pa... - Nuckyz
9cada9a fix(Webpack): Not canonicalizing regex in some ... - Nuckyz
904022a [Webkeybinds] Don't override browser keybinds (... - rniii
3d80cb2 Delete AutomodContext ~ Now a stock feature - Nuckyz
66b247b Fix persisting $$vencordPatchedSource when a mo... - Nuckyz
1.9.8 - the nuckyz update (ft. rniii)
NoMaskedUrlPaste ๐ญ
We had to make a plug-in to stop the feature that was a plug-in that discord stole
scared me for a second
sorry i ate the object
that means u put something in settings that isnt plain object
what did you do
no idea I didnt even touch more user tags recently?
why did it turn that into a settings proxy
^^
happened in the past already but I had fixed it
@limber skiff on vesktop, with volume booster disabled, i can fully recreate it
if you're experience something else, could you please send me a video so i can understand what you're talking about
what ๐ญ
actually I might know how to reproduce
I had changed a more user tags setting when that happened
yep
happened
thank god
๐
@austere talon IT HAPPENS IN DEVVVVVVVVVVVVVVVVVVVVVVVVV

?
the error
is not only in my branch
it might be only when you change more user tags settings, but i'm not sure yet
oh yeah
i can repro
ya
seems to be more user tags
yeah cause this is proxy
and it's calling setValue(proxy)
I think the issue is just that's it's a nested object
how was this never caught
no
that should not be using store
it should use plain
or clone the object itself
let me test something
that code is just bad
okay yeah
it should use plain not store
but that will make it not reactive
so instead it could just forget about the save button and never call props.setValue
MoreUserTags is the only plugin that has a nested object in settings and the settings UI modifies it
once again that's not the problem
I know
the problem is with the plugin
how it was implemented is the problem
I'm just mentioning that other plugins that have a nested object could have this issue too, but they dont use the settings UI to modify it
ignore activities for example
no they cant
they could if they implemented it wrong like more user tags did
but whatever it's just an issue with it as you have mentioned
either use plain and handle state urself
or don't call props.setValue() ever
both will work
that code is just wrong
i think we should get rid of the temporary settings and the save button entirely
have the settings be live
I had done this
@limber skiff I was able to fix the stock issue by removing the math.round.
feel kinda stupid now for not thinking of that earlier
while it is accurate now, it is not stock behavior with volumes lower than 100%
won't update the ui im pretty sure
cause you're not changing any state
it is
const [tagSettings, setTagSettings] = useState(settings.store.tagSettings as TagSettings);
that's the state
yes but
setTagSettings(tagSettings)
this does nothing
react ignores that
if the new state is equal it wont change
you would have to create a new object to cause it to rerender
like
onChange={v => setTagSettings(s => ({ ...s, text: v }))}
anyway that code was just bad
that entire state stuff made no sense
I'm just gonna port settings.plain since it may be useful in the future
what do you think about this though
personally i can't think of a scenario where i changed some settings and then want to reset back
and it makes our logic more complicated
and rn it has some issues with validation where you need to reopen the modal after changing some setting
even if there were, it's easy to just manually change it back
oops stole your commit
honestly entire settings ui needs a refactor anyway, it's so convoluted
we just kept adding more and more to it and now it's so messy
I agree
it could benefit greatly from being split in more smaller components in separate files
rn it's a nightmare to change stuff
xd i dont care
i should cherry pick some of my changes from asar pr
i still wonder how it wasnt caught until now
ikr
oh u added plain
i was confused
anyway if u wanted smth to work on, maybe you could do auto reporter instead, ifeel like you might be interested
essentially just whenever new canary/stable release run tester
like we talked about before
i haven't done much so far, only detection for new version (which polls latest.json once a minute)
(if you're gonna finish it, you can choose a different language too if you want, i only did go cause i like it)
is it also like this for u
it's not
so maybe discord web or vesktop bug
anyway feel free to finish it if u want to
(but only if you want to, it's more a "hey here's this cool idea for you" than it actually needing to be done)
i started on it but then kinda lost motivation
and much other stuff to do xd
it would certainly be useful tho and help us catch errors literally immediately
yeah for sure
btw i forgot, does reporter even still need login?
if it works perfectly without being logged in, we could run it on prs
let me see
cause then they can't do anything bad
worst they could do is get the github runner rate limited from discord or smth
looks like it works and doesnt even produces non verified account errors lmao
(ignore the log I removed the login line)
say what lol
that u should remove token from validation
oh lol
None
None
None
None
so yeah we can run it on prs then
None
None
None
None
buddy what
but we have to make it so that only the touched plugins are enabled
it made it so much slower
are u sure
nvm
btw you pinning the reporter action ruined my muscle memory
17 hours ago

might just be that github or discord is busier rn
gonna re run on old commit
None
None
None
None
None
None
None
None
this is joking with me
now im gonna re-run latest commit and it's gonna be a turtle just watch
ShowHiddenChannels (took 3.1000000000058208ms):
ID: 473403
Match: ```
/.name),.{0,120}.children.+?:null(?<=,channel:(\i).+?)/
None
None
None
None
None
None
None
None
None
None
None
None
None
None
None
Added the plugin to mentionAvatars as requested, it still works the same way (for both), but i couldnt figure out the patches to create an setting option to disable "@" before role mention
And i dont know if we are keeping the plugin name the same
@keen knoll your plugin constantly errors, please for the love of god fix this
just use the store
dont spam discords api
i might rewrite the whole thing 
Adds new option to NoBlockedMessages, default off:
Without on
With on
!Discord
!ShareX
!ShareX
true
i will make another PR for a plugin I already contributed to
Renames ShowTimeoutDuration to ShowTimeoutDetails and adds reasons for timeouts.
Opening as draft for now as I haven't looked at the UI for automod timeouts properly and documented what it looks like here
wrong channel
why was ShowAllRoles removed, if there is no default way to have it expanded (to my knowledge)?
someone definitely was not yapping in the wrong channel about it btw
People are complaining in #๐ฅ-vencord-support-๐ฅ about this now
I wouldnโt say its complaining, more asking and saying they miss it
๐
True
Issues should be resolved
are you sure this !important is required? you already specify 16px in the attribute. pick one
Is there a possibility to show the icon on the left like avatars, and replace the mention? Disclaimer: I haven't personally tested the code
do not replace the existing screenshot with a new screenshot that doesn't show the original functionality
and replace the mention?
What do you mean exactly with this?
why dont we combine all this stuff into 1 plugin?
what do you mean?
the discord and server join date
yeah because of proxy
Seems it still pastes when releasing middle click if you hold it too long
That was the idea and will probably added i just couldn't figure out how to
width: 1em;
fine with this?
I'm sorry, I don't quite get what you mean
I didn't replace it, the original one is still there
None
None
None
None
None
None
None
None
Added a 0 option to the slider, and a check that returns false if the limit is 0.
Is that just what they use for the settings info box?
it's just hardcoded there
this one also returns the build number as number
I also have the same issue. Can I help to resolve this somehow?
The plugin is currently completely broken. Wait for a fix.
i imagine its just inlined
ye
Removes dependency on virtual-merge, this was only "required" by fakeProfileThemes, however you can just merge objects normally in this case, without worrying about reflecting the merge back onto the passed in user parameter. Will add myself as a plugin author if needed/wanted, but this feels like a simple enough change to not need it :3
I also feel like nanoid is a pretty somewhat dependency here, unless it's planned to be used in more places than just the notificationLog (maybe a vencord api to generate a random ID?)
lmao the downloads for this package spiked when ntts made his video lol
lmao
these are such small packages it doesn't matter
tbf i dont understand why virtual-merge was needed
they are right in this case
nanoid though is fine to keep
its tiny
isnt it like 130 bytes or something
unless im misunderstanding and you want changes to be reflected back to the original
though it is very slow compared to crypto.randomUUID, however its not frequently used enough that this would really matter
nah
such a minor change should just be tucked in with some other pr
crypto.randomUUID 7,619,041 ops/sec
uuid v4 7,436,626 ops/sec
@napi-rs/uuid 4,730,614 ops/sec
uid/secure 4,729,185 ops/sec
@lukeed/uuid 4,015,673 ops/sec
nanoid 3,693,964 ops/sec
oh my god it is FAST
girl this won't be called more than twice a minute it doesn't matter ๐ญ
the only advantage nanoid has is that it has a bigger alphabet but for our purposes this doesnt matter xd
nah its just cool to see
having a native uuid library thats faster than the third party ones
also dont mind me, ive gotten really into performance optimisation lately
because thats what im employed to do atm xd
What is currently needed to make this PR happen? If it's within my capabilities I'd love to help.
return Object.assign(user, { premiumType: 2, themeColors: colors });
you'd wanna return the same user object still, this is being passed to discord code so not many assumptions can be made
this even still is probably not a great idea, i think virtual-merge is a fine solution
this was deliberately done to keep proper expected behaviour. namely, spreading turns the object "read-only" (since it's now a copy) and breaks class instance fields / methods entirely. but we also don't want to pollute Discord's objects with fake properties because it might break things
it's also such a thin dependency that it has very little impact
yap

Wouldn't using Object.assign prevent instance methods from borking here? In my (albeit limited and a bit silly) testing, this does shave size off of the bundle and make it work ever so slightly faster
but we also don't want to pollute Discord's objects with fake properties because it might break things
It doesn't look like the thing that this is patching actually uses the user value that's returned for anything else though, so would this still pollute even if it's essentially getting immediately discarded after render?
I thought with virtual merge you'd be reflecting the changes to the user object, which sounds like what you don't want
i think there's little reason to add it to role mentions considering their icons are at the back which doesn't "conflict" with the @
this seems very overcomplicated. match as little as possible. you can use arguments to get the argument without explicitly matching it
I could be confused here, but
However, changes made on merged will not reflect back to the original Objects. This might be desired, in which case sweet, you don't need this module!
Makes it sound like any time you call virtualMerge, the changes are returned and it mutates the first parameter
why not just reuse the other class? considering they're almost the same
renderRoleIcon: ErrorBoundary.wrap(({ roleId, guildId }) => {
please format your code better. keep lines a sane length and use newlines. you also seemingly haven't properly configured your editor
const obj1 = { im: "so cool" };
const obj2 = { hamburgers: "are delicious!" };
const merged = virtualMerge(obj1, obj2);
merged.im = "so cute!";
console.log(merged.im); // "so cute!"
console.log(obj1.im); // "so cute!"
Yeah that's the example I was looking at.
Reading through the code for virtual-merge, it looks like it just proxies every property that it can? Isn't reflection in JS horribly slow at times? 
besides that, again why would a link between this new user object and discord's user object need to exist if it basically gets immediate discarded after render? fakeUserProfiles should be overwriting keys that already exist, instead of making new ones
isn't it just setting properties that would already be there?
no
you're not understanding but im too lazy to explain
the way it is is good, just leave it unchanged
also virtual-merge is actually tiny but wanting to clean up bundle size is fair
u should be looking at the esbuild analytics thing tho
esbuild has a visualizer for this
i think my shiki plugin takes up like 30% of vencord lol
yeah im aware, it just seemed like an easy cleanup, and for keeping functionality it literally was a one line change
Yeah I saw that shikicodeblocks was huge lmao
Also another thing I mentioned in the programmer channel is that vencord contributor icon fetching in the userscript distribution is broken because of CSP
there's two options basically
- embed the image blob into the vencord bundle for userscripts
- use GM.xmlHttpRequest
2 is probably better, but im not 100% sure that gm can handle non sync xhr
what about the secret 3rd option
where we maintain a webrtc http proxy
bypass all CSP
i might actually genuinely do this for my vencord fork
also yes webrtc has literally 0 cors or csp or anything at all
Boy do I have the semi convoluted not very well suited for this solution for you 
theres a csp rule for it being pushed by w3c but it really looks like chrome doesnt wanna add it
Does the websocket ctor have cors also? I've never checked
yea i found this while researching how long i'll be able to use WebRTC in discord activities
But I've also never encountered cors issues with websockets
this makes me think pretty much forever unless discord adds it in their electron fork or something
or chrome team has change of heart
does this mean constructor?
it doesnt matter much
badges are all broken
I mean if they haven't implemented it in 8 years, what's the chance they're gonna implement it any time soon
no way to fix it without awful workarounds
people should just not use the userscript
gm.xmlhttprequest and blobbing the response?
That doesn't seem super awful to me
does that entail making a fetch-ish util that uses a buildtime constant for impl
wisp protocol..,...,,
lol what the hell is this https://github.com/MercuryWorkshop/wisp-protocol
we alr do this
userscript build "polyfills" window.fetch
with an impl based on GM_XMLHttpRequest that bypasses csp
anyway there's little reason to do it for badges
does that work with manifest v3 userscripts
actually of course it does theres no csp in background scripts
speaking of MV3
we should migrate the firefox ext to mv3
it supports it since 128
thank god we arent gonna support extensions in our fork
the extension build isn't super practical from what ive seen
but the userscript build is kinda cool
wdym not super practical
it's mostly what I use for testing since it's the easiest to just load into the browser for me
oh
ive seen very few people actually use it
i mean...
so like not super common?
extension is the main build
oh u mean ok
only benefit of userscript is more frequent updates
fair
are there any current outstanding issues with vencord or vesktop that nobody has fixed yet
i want to contribute and it's easier to ask if there are any big issues right now
yes
idk what vee thinks is most pressing but imo
i think its that we need testing for the new updater system
i gave up on it but if someone else helped push along https://github.com/Vendicated/Vencord/pull/2707 that'd be cool
cus i'd still like to add plugin HMR
I can test out building it on linux but i assume that's where pretty much everyone is devving on lol
yea we need windows testing
there were some issues another vencord dev ran into
i'd go fetch message links for u but ur not on windows so it prob doesnt matter lol
yeah I'm not even 100% if my windows partition works anymore 
real, same happened with my arch partition
this can't really be added in its current state because this is sandbox escape. it needs to be fully controlled by the native process, eg by making this ipc method take 0 arguments and then instead prompting the user with a native OS dialogue
anyway i'm unsure if this is even a good idea
we don't want normal users using this
people should just edit the patcher.js file to change the repo
if you have an api like this in the bundle everyone uses, there's a 100% chance someone will eventually find this and then advise non techy people to do it to change to dev branch for faster fixes
and then we get people asking for support with the unstable branch which sometimes has broken code and frequently gets force pushed which will break the updater
yeah, you could give a warning or something like "we will not give support if you use this" maybe
You're overestimating reading comprehension
yeah excluding it is probably the best idea
See also: "why did ThisWillBreakYourDiscord plugin break my discord ๐คช"
Did someone actually ask that in support
No idea, but probably
Someone needs to invent a captcha but instead of testing if you're a robot it tests if you're an idiot
Isn't that just fn is_moron(&self) -> bool { 
}
Fixed
thats why theres no UI
people will still use it ๐ญ
idk any way that makes it easier on us devs that normal people wouldnt be able to do if given instructions
but yea u cant change repo
this
real if u wanted to yea
not really needed cus u can just use ur own repo it'll be the same if its a fork
then why have the vesktop optiosn
theres no 'vesktop option' in my PR
because otherwise there's no way
nono vesktop has an option to change the vencord source
but like why make it a user accessible ui instead of a build option
vee just prob wanted something easy for them
because you dont need to build vesktop from source to change vencord
oh yea theres no patcher fuckery
you just said u dont want normal users to change vencord repo tho ๐ญ
i mean if this gets merged you won't need that ui anymore i think
cus u just replace ur build with the devbuild from ur fork, and switch to branch-thing target to test updater
@austere talon do u want me to add a package.json script for downloading build artifacts to ur install folder
actually no thats unnecessary u can just build them urself and copy
actually i might need to fix build scripts before u merge
i know the git updater doesnt actually check what repo ur on, it just assumes ur on origin, so if build script does the same i need to change
Is it even possible to move them to the left, ive tried so many times
Discord Account
nedelio
What happens when the bug or crash occurs?
If you go with your mouse over the toggle button over and over again fast, like up and down, up and down, then the message when you hover over it (Disable Game Activity or Enable Game Activity) sometimes bugs and stays even tho you mover your mouse from it and you can go into settings and it will stay hovering there until you hover over the toggle button again and make it dissapear.
What is the expected behavio...
Thatโs probably a discord bug ๐ญ
โWhy does this bug happen when I do something like 10 times really fastโ
Doubt it
erm
maybe
wtf is that
something has been oversimplified here for sure
oh i see now
i totally see now
this is bypassing React.createElement
that's almost certainly part of the reason why it breaks
whenever I see that there's sometimes unusual behaviour
no it's not
it's jsx, vencord turns it into a createElement
all vencord components work like that lol
does that really Just Work
wouldnt the hook be 1 level above where you'd ideally want it, though that shouldn't matter too much
ive only ever seen the tooltip get stuck on the game activity toggle ยฏ_(ใ)_/ยฏ
i wonder if it has anything to do with the lack of a key there
maybe the function breaks the auto generation of keys
idk though
hmm theres an ErrorBoundary wrapper though
this is the only place in vencord I've ever seen tooltips break like that
where it just Doesnt Disappear
for some reason
does this even matter
What the fuck?
maybe, just not as much as I might think it does
this needs to be refactored (in some way)
okay this is definitely its own thing its just super confusing looking
what's wrong with that lol
you'd think it's the stock Button component when it isn't
it's the one used for the buttons in the side of it
does it happen with the stock buttons
I know it happens a lot when I switch to another windows moving my mouse down while passing above the activity button
stock buttons dont implement this tooltiptext thing
it's right there
Weird that minifier picked different minifications in different cases
there is nothing wrong
(I'm not being ironic)
That looks like completely normal minified code, I don't know what you're confused about
yeah
ngl minified code actually made me start to use == null and != null almost all the time
Instead of what?
I'd just do h?.()
!= null only targets undefined and null
yeah I know
but I mean for other cases where you cant do that
Considering the amount of code I've seen that does a != null && a != undefined, I don't think == null targets undefined
Right
it's why the minifier uses != null there
either the original code is d?.()
or if (d != null) d()
because there is no way it would transform if (d) d()
to that
Yeah that'd be d&&d()
yeah
definitely ?.
definitely !
definitely !?.
@austere talon
where all does the build process use/embed the repo
oh yea the ~git-remote plugin my bad
@Vendicated can you look at this commit I just pushed: 90a6063 (#2772)
Let me know if you think I should do this in the git updater too or something
None
None
None
None
None
None
None
None
Im tired and youre asking for a review, so i gave it to ya lol
just verify everything is accurate before changing stuff.
i think youre meant to use a class name factory for this.
shouldnt this be addCopyButton?
not certain.
I dont think youre supposed to put spaces in replaces
replace: "$&self.AddCopyButton({$1,$2}),"
verify that works, im not sure, and im tired
this was deliberately done to keep proper expected behaviour. namely, spreading turns the object "read-only" (since it's now a copy) and breaks class instance fields / methods entirely. but we also don't want to pollute Discord's objects with fake properties because it might break things, so we can't just write these properties on the original object
it's also such a thin dependency that it has very little impact
I don't see why the properties can't be set on the [UserProfile](htt...
Class name factory seems to be for when there's multiple class names all prefixed by vc-cfc- (cfc in this case). So I don't think I am supposed to use it for this.
hmm... if only they thought to put the reasoning in the commit message....
Because pressing a single button is somehow too much for people
#๐ฅ-vencord-support-๐ฅ message
pretty sure they meant "why? add this back"
Hello, I'm not very knowledgeable about github, so what about this plugin request? Was it abandoned or done? If this is done, where can I find the necessary files? (google trnslt)
horror
jumpscare
laydowncalm
None
None
None
None
None
None
None
None
Slightly changed the code, here's the result (for automod)
Also the icon is orange when automod does timeouts
wtf are you trying to do
discord api is cursed and returns a string of one of these values in audit log responses
i will implement later

Hide the effects in profiles, as some of them can be very overwhelming when viewing a profile.
this is a glorified css snippet
[class^="profileEffects_"],
[class*=" profileEffects_"] {
display: none !important;
}
Into quickcss.
wait it's actually just css
yea lmao
i just copy/pasted what they put in the plugin and told em to put it in css
users yearn for a css snippet/theme store
if i make a plugin with a collection of css snippets, would that get merged?
why would it
thats something called a "theme"
themes dont have individual toggles for each of the things they do
you can release a series of imports that enable/disable specific snippets
because its been mentioned before
?
i thought there was a usercss pr for that
we had a thing for that, its called #๐จ-css-snippets
@turbid hatch iirc you were working on it?
yes but it hasnt been worked on
I NEED TO TEST
none of the buttons work yet, but the ui does
this is the second time Iโve been sitting near my parents watching a video that you sent and it says the n-word
other then that Nice
also steam jumpscare
wtf
FREE SMILEY DEALER IS BACK
try sending a ๐ญ
๐ญ
๐ญ
Adds a plugin to always expand the role list in profile popouts.
This was highly requested and sparked drama in the Discord server, very small plugin, not sure what else to say.
not sure if thats a good match, tried my best lmao
fuck forgot to lint before push
honestly
just change the default expanded state
so it's expanded by default but u can still minimise it again
it opens as if you clicked the +
oh wait you can minimise it again? ๐ญ
i did it like this cuz it was easier and i didnt know it would get rid of minimising
ill change it then
the collpase roles button in question
there we go
now just expands by default
also fixed checks
๐ญ
niche feature imo but if so many people request it
shrug
is it actually that many people
well yea
rmber vencord has hundreds of thousands of users
more than with any other request
technically yea but a lot of people also just dont bother requesting it or something but i mean it doesnt really hurt including it now that i made it right? :3
i might need to cut my leg off to merge it..
i am making that 21
We should probably migrate the settings from the old ShowAllRoles
people who had it enabled will likely already want this enabled
how would we do that
alright will push then
the plugin also needs a README
right
girl what is that font
its right there xd
I CAN TELL
another fun fact it looks even worse in the terminal because kerning is disabled there
We should probably migrate the settings from the old ShowAllRoles
people who had it enabled will likely already want this enabled
Added, also added the missing README.md
there we go
10/10 id use it
How about adding an option to hide the collapse button?
why ๐ญ i think we do not really need to add options for such a simple plugin
How about adding an option to hide the collapse button?
could you just use CSS for that?
wouldnt hiding the button just bring back the plug-in that this is meant to replace?
yeah i was about to say that
what
fuck you autocorrect
ah
well shrug
if you want it to be like the old plugin, as @sadan4 said you can use css
exactly
every time someone says "use css" a little bit of #๐ฅ-vencord-support-๐ฅ enters my soul
I would agree with you, if it wasnโt to hide a single button at that point yes use CSS
me when i dont use
#1257070841372610581 message
also, you could just be like this person and ask in support like a normal person
#๐ฅ-vencord-support-๐ฅ message
whenever i import from plugins in my file, i get this error
what am i doing wrong
recursive imports
is there a simple way to fix?
use Vencord.Plugins instead
thanks
None
None
None
None
None
None
None
None
green 
you donโt have your server role on this new account?
was never given them upon request
๐
For all of the people wanting the plugin without knowing how to compile Vencord, you can always use a Vencord fork (unofficial and unsupported edited version of Vencord). Vencord will be available prebuilt with unapproved plugins. Just know that using a Vencord fork is at your own risk, as the plugins in it might not properly get checked for malware and plugins might stop functioning properly at any time. Anyways, if you really want the plugin, that is an option and there are some good Vencor...
Can you stop advertising the forks?
For all of the people wanting the plugin without knowing how to compile Vencord, you can always use a Vencord fork (unofficial and unsupported edited version of Vencord). Vencord will be available prebuilt with unapproved plugins (like this one). Just know that using a Vencord fork is at your own risk, as the plugins in it might not properly get checked for malware and plugins might stop functioning properly at any time. Anyways, if you really want the plugin, that is an option and there ar...
thank you nyx
No matter how many times we tell people not to go talking about the forks in the official repository, they do it anyways
@limber skiff would it be possible to still set $$vencordOriginal even if a patch errors?
why
to diff patches that error
you mean the patched source?
yea
hmmm
unsure
a successful patch after the error would overwrite the errored source though
since it would use the old code that doesnt error + the patch that succeeded
yea, i think ill just manually apply the patch for diffing an error
we can maybe create a set/array with the errored sources
all i need to do now is make the jump button
they broke permissions viewer again nice
@rugged spire you love
why lmao
idk because itโs different for my own
if (typeof Object.entries(members).map(member=>member) === โstringโ) return true;```
Smart quotes found ๐ซ
what is that
my favourite go snippet
This plugin hides the newly added Activity Tab on the members list.
whats with all the css snippets lol
.content_eed6a8 .container_c64476,
.membersGroup_cbd271:has(> .headerContainer_bc6acb) {
display: none;
}
I am aware that theres the option to hide it, but the label still remains there. What this does it completely hiding anything related to that feature.
I am aware that theres the option to hide it, but the label still remains there. What this does it completely hiding anything related to that feature.
so use a css snippet lol, whats the point of a plugin, that literally IS just css
I am aware that theres the option to hide it, but the label still remains there. What this does it completely hiding anything related to that feature.
so use a css snippet lol, whats the point of a plugin, that literally IS just css
well i am using it, i wanted to allow others to disable it if they wanted, without having to write the css.
Then post it in the css snippets channel in the Vencord server.
@Vendicated hello, just a reminder
mods, lobotomise this user
๐ง ๐จ
mods, lobotomize me
vee, close his pr
Using the GameToggleActivity button code I was able to add a similar feature to CustomRPC, where before this, you would need to enter the plugins page and disable it from there, now, all you need to do is click on the "Disable CustomRPC" button added to the account panel.
This change is way to niche, no one really needs a shortcut to enable/disable CustomRPC, additionally this clutters up the profile space and even breaks without a theme:
Why was this closed????
prolly this
can you unlearn the massive margins thing im begging you you're not a megacorp
no
i will margin you
if you dont like the UI, make it yourself
?remindme 1h try dev companion on my laptop
Alright @rugged spire, in 1 hour: try dev companion on my laptop
@rugged spire, <t:1724876884:R>: try dev companion on my laptop
i think its time
I get it, but moments like this don't look good
- Instead of broadcasting in the profile banner, I'd leave the voice channel icon, just with a different tooltip
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
**__WebContextMenus (had no effect):__**
ID: `343602`
Match: ```
/\i\.isPlatformEmbedded/
WebContextMenus (had no effect):
ID: 343602
Match: ```
/\i.\i.copy/
None
None
None
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
None
None
None
Console janitor mid anyway, my version better
What changes does your version have
why is the patches broken on stable and not canary
normally it's the other way around
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
**__WebContextMenus (had no effect):__**
ID: `343602`
Match: ```
/\i\.isPlatformEmbedded/
WebContextMenus (had no effect):
ID: 343602
Match: ```
/\i.\i.copy/
None
None
None
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
None
None
None
Always expands the role list in profile popouts
match: /(roles:\i(?=.+?(\i)\(!0\)[,;]\i\({action:"EXPAND_ROLES"}\)).+?\[\i,\2\]=\i\.useState\()!1\)/,
replace: (_, rest) => `${rest}!0)`
Sorry if this regex looks like hell, it's just abusing lookaheads to make sure we are changing the correct useState
find: 'action:"EXPAND_ROLES"',
How about adding an option to hide the collapse button?
not worth that's so minimal
description: "Max Super Reactions to play at once. 0 to disable playing Super Reactions",
pr review jumpscare


in fact i'm gonna include the other one too
it's not used in replacement but in the match
you have a typo in your suggestion
where?
nono
oh?
reference to second capturing group
ill explain in a bit
they make the match super unreadable
can you figure the magic
here's the hint
(roles:\i(?=.+?(\i)\(!0\)[,;]\i\({action:"EXPAND_ROLES"}\))
I LOVE REGEX BACKREFERENCES
does the mobile app have something similar to user voice show?
did you enable this
yea i did
i was concerned specifically cuz it didnt show as working on web for me
but its whatever now ๐ญ
thanks!!
as scary as that regex seems like it actually should not break easily
in fact super reaction tweaks has on similar
same concept
There we go, checked through the reviews again and applied the changes
ah, alright!
still getting into patches :3
no worries it was good
it's just that modifying useState sometimes can be very hard to be specific and you are forced to use complicated techniques if you want to be sure of what it's patching
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
None
None
None
ConsoleJanitor (had no effect):
ID: 316792
Match: ```
/,"notosans-.+?"/g
None
None
None
rip ConsoleJanitor
notosans error is gone
chain breaking image
AlwaysExpandRoles is fire
pushing to main
:3
None
None
None
None
None
None
None
None
7ec842d MoreUserTags: fix settings ui logic - Nuckyz
4b16fbc MentionAvatars: Also display role icons in role... - SerStars
7d8214f Fix PermissionsViewer on user popouts - Nuckyz
5bfc608 new plugin AlwaysExpandRoles ~ Alternative to S... - surgedevs
db2f5c9 ConsoleJanitor: Remove non needed patch - Nuckyz
what's the worst that can happen, right?
while you werent looking, i exploded a patch
Canโt wait for the announcement thatโs never gonna happen in #plugin-news about this new plugin
wtf am i cooking?
@rugged spire you will make setting pop out to external window
like you can do with vc's
what
that's pretty cool ngl
toolbar is where elements like these are defined
hot take
GuildSettingsAuditLogStore should be rewritten by Discord
its such a bad mistake
guhhh i do not want to write AST parsing code at 3 am
it is a Store being treated like a Cache for the current audit log view
!remind 8pm write PluginDefCodeLensProvider.ts
Alright @fossil inlet, in 16 hours, 43 minutes and 42 seconds: write PluginDefCodeLensProvider.ts
honestly I'm not sure if that's weird or not
maybe it's good for use with useStateFromStores
I wonder how many people peek in this channel daily
in powercord ages when I didn't do dev I used to peek the development channel everyday to see if there was interesting stuff happening
any new messages were a great time for me
there are probably a couple of people
and then those who'd love to participate here but dont know how to and are too scared to ask
ive lurked in random channels from time to time
I just now realized the amount of reactions in this message
a few
is that channel dead..
Most announcement channels are
it's not dead, just there hasn't been any new plugins merged
Hasnโt there been a release since that last message
releases are just for the browser extension
yea about 20 mins ago :3
oh
you mean version release
they are not the mark for new updates
im stupid
main commits are
actually this should be announced
and AlwaysExpandRoles
if vee doesn't do it soon I will do
Here's another example. I am seeking feedback on this now
Could this be improved?
it is
so imo its fine
honestly the issue is not that it's a popout
because they can look good
maybe it just needs a better design instead of being all text
hm maybe
ironically this popout has horrible UX
I like it
this is how it renders 8 people (that's 2 voice, 3 streaming, 3 playing games, 2 in activity)
Discord recently updated the activity cards on profiles to a new design, and changed how the Spotify activity appears. It is no longer an optional feature of all cards, and is a separate card.
This update adds the time bar component to the new standard activity cards if they have valid timestamps.
It also adds options for hiding the "badges" on the cards (game controller/tv with the elapsed time) and the large_text attribute on listening activities, making it look like the Spotify car...
that reminded me of this so pretty please merge Vencord | PR #2712
Alright @limber skiff, in 10 hours: merge that
Yeah, itโd probably look better if they make it wider than like 2rem
@limber skiff, <t:1724931090:R>: merge that
will they though
when I actually wake up
what tz are you in
new feature!
I think this will be better
some fucking changes included, will be reposted
i think more servers will use this permissions schema and i think will be better
description: "Can timeout, kick or ban members.",
remember that these are shown as all caps so keeping a space between both words is preferable (also keeps consistency with vc mod)
donโt add yourself to devs for minor changes
if everything youre doing ends up being usable, ill probably switch over to vscode
what are you using now?
I LOVE JETBRAINS I LOVE JETBRAINS 
use notepad++
i used to use pycharm but i dont use python enough now to warrant
does npp have lsp support?
(and rider i switched over to visual studio)
lemme just fire up the apple notes app for my code rq
what would you rather i use?
something that you dont have to pay for
i dont pay for jetbrains lol
its paid tho
(ignore deleted message, i forgot theres no community edition of webstorm)
@fossil inlet, <t:1724915778:R>: write PluginDefCodeLensProvider.ts
I'm (slowly) working on an intellij port of the extension, I assume it'd work on webstorm 
Work do be getting in the way though
im curious how far you've gotten tbh
seems hard to completely port it
esp the webview
and the Typescript AST parsing
Pretty much just the lenses for testing patches/viewing modules and diffs
I had to set it down for a bit because I was spending too much time fucking around with it and not doing my actual job lol
yess
i might look into how extensions are made and try to contribute, is the code for it on github?
None
None
None
None
Settings (had no effect):
ID: 394644
Match: ```
/({(?=.+?function (\i).{0,120}(\i)=\i.useMemo.{0,30}return \i.useMemo(()=>\i(\3).+?function(){return )\2(?=})/
None
None
None
not a problem
