#πΎ-core-development
1 messages Β· Page 201 of 1
@austere talon started
restarted
unironically a good idea just make sure to cache them
@austere talon i just realised something, does asar-bundle have to do anything with my so-called skill issue i had yesterday where loading @sentry/electron into cache didnt work for me what I think is that you are rewriting loads of stuff just to make getting rid of that easier? is that right or am I crazy
whar
tl;dr: are you moving vencord files into asar archive so we can bomb sentry without horrible code or is it just because its a good idea to do it this way
lmao
no correlation
alright
just curious though
why do you want to move all the vencord files into an asar archive
its real π
Wtf is ventord
I can see one person I know using that plugin
istg if I get a ping telling me to add it 
who π

not even a second
speaking of which time to request a plugin

vencord users on their way to make plugins that may or may not add slash commands that immediately send NSFW content:
i dont even think im safe from that one fuck you @still wasp (for actually making my stupid idea) 
?remindme 1h revive GifRoulette and make it even more obnoxious
Alright @rugged spire, in 1 hour: revive GifRoulette and make it even more obnoxious
real
real
You asked for it fuck you
You ask and i deliver
I will turn Equicord into the Femboy fork
I can not be stopped
sam cute when mad
@rugged spire, <t:1721336719:R>: revive GifRoulette and make it even more obnoxious
no
for localization
so strings can be packaged
but
there is no localization api for vencord yet
is that being built along with this?
forgot it was restricted
it was open to viewing previously
no access my beloved
Β―\_(γ)_/Β―
vee took our ability to edit channels away so cant do anything about it
for security?
i can let you in if you plan on contributing to localization
yeah
we didn't really need that permission in the first place
how many things is there to translate
i'm not really suited for providing translations but i might be interested in how the system works, come back to me about it later
honestly idk
oh
i do have feedback though
all the ui
does this include plugins?
maybe don't squish everything into one source file
i think its split per plugin
the only proper way
does this include setting names?
π
everything except plugin names iirc
we're only going to support the languages that discord does
separate translations folder is a bad idea imo
latvian is in there
cool
really bad for dev UX imo
then ig just wait
jumping outside of your plugins' folder to write text seems kinda bad for dev workflows?
can you partially expand a folder in vscode (no?)
its better for the localization software we're going to use
easier to have them all in one place
what software did you decide on
you can also just open it as a second file
ofc
crowdin
any reason why that was picked compared so something else
its ai powered π₯ (trust)
no selfhosting
wtf
are you saying that is a selling point
60,000 hosted words π
being forced to selfhost is a bad thing
oh
there are hosted alternatives 
ik about weblate
yeah iirc some of the other devs had issues with it though
lewi had their own reason for crowdin i just can't remember
nah sounds like "let us use your project as AI training data for other people to use"
i think it was about a specific string type or something
j
im going insane
love
how lovely
shrodingers asar file
π₯
I build the asar branch at the j commit and relaunched. Got an error probably becuase I'm using the git updater. Went back and built main branch and relaunched,, now my localstorage is resetting every launch lmao this behaviour is wild
what error?
it should work fine
oh nvm
you need this
I think it was like
duplicate listener
yes
thank god
it was cause I was accidently importing http updater even in git updater builds
which led to duplicate handlers
is that on windows?
yea
can you try with normal install
pnpm build --standalone then copy all files from dist/desktop to AppData/Vencord/dist
and launch Discord
it should migrate and relaunch and work
so
oh with normal installer first ok
seems like that worked
it did something before it actually opened a visible window
oh probably made vencord.asar
it downloads vencord.asar, writes this to vencordFiles/patcher.js:
// Legacy shim for new asar
require("../vencord.asar");
```then relaunches
yea worked
can u verify that patcher.js indeed contains that
yea it does
fire
now just need to test updater too and also workflows
and then update installer and vesktop

alright
working on this rn https://github.com/Vendicated/Vencord/pull/2707
yep I saw it
is github ok
https://github.com/j4k0xb/Vencord/tree/block-keywords/src/plugins/blockKeywords
edit: deleted fork, waste of time trying to contribute
Huh? Why though? Can you elaborate please? @j4k0xb
okay it works
pnpm tauri
amaze balls
wha
okay now i only need to test on windows and also test the updater
then make release of installer, vesktop and vencord

i will do tomorrow
so if soimething goes wrong i have plenty of time
imagine if i push this rn then go sleep
stuff will explode maybe and average support user will die without their vencord
pushing big update on a friday
you'll do great if you get a software engineering job
Please do not private message Vencord plugin developers for support!
Instead, use the Vencord support channel: β Vencord Vendor Venniecordβ π₯-vencord-support-π₯
This warning should not show up if you have a friend request from the developer
irrelevant
i disabled that in my build 
it agrees with this sentiment
its weird when someone friends or DMs you and then your client tries to tell you off
also just accept the request or remove the warning
I should be able to meow at people and not have them be yelled at for looking
the message is prob for the better tho
horro
OMG
okay this should work
(use case insensitive)
17E3 is also valid
cause
it doesnt work inside string
π
its a literal string
Number("17e3")
π
discord
amzing
None
None
None
None
FriendsSince (had no effect):
ID: 926150
Match: ```
/USER_PROFILE_MEMBER_SINCE,.{0,100}userId:(\i.id)})})/
None
None
None
it's rspack which uses swc
but i think any minifier will do this
I'm surprised this never happened before
oh i see lol true
it's fairly unlikely for ids to have 3 consecutive zeroes as ending ig
Discord Account
CJStylesOrg
What happens when the bug or crash occurs?
the roles will not show up on the mod view at all despite the user having a role shown in their profile. i have tested this on Bread (304260051915374603) whom is a plugin developer in BetterDiscord. cases like this are rare but i managed to stumble upon this.
What is the expected behaviour?
i expect the plugin to...
discord doesn't show the highest role there by design
we need a Inject profile data feature for this at this point π
I maintain a similar plugin
also broken again
π
ignore that
nvm i'm wrong. This is a stock Discord issue that we can fix (and need to fix). Discord is using MemberSafetyStore.getEnhancedMember(guildId, userId) to get the highest role, and opening Mod View from the context menu does not try to load that information. See attached annotated screenshot for what needs to be changed to fix this.
![Diagram showing to remove MemberSafetyStore.getEnhancedMember and related get role call and to just find the highest role from the member's (passed to the co...
tl;dr: discord overcomplicated things and depends on stuff that isnt important at all here
Did some testing and I can confirm the updated does work!
I'm never getting the time I spent porting stuff because of vesktops downloader hating asar files
Note: This is technically a stock Discord bug that this PR fixes. ShowHiddenThings just makes this bug easier to be found (by making Mod View used more often).
See https://github.com/Vendicated/Vencord/issues/2708#issuecomment-2238512347 for how this works and why it's needed especially with ShowHiddenThings.
the commit in vesktop is broken
i already fixed it but I didn't push that
π
looks like u cant really reload asar
so in dev it would have to build non asar
This plugin automatically updates your status when playing games and reverts the status back when the game is closed.
this looks too selfbot-y
also RPC
am I missing something about the rpc part?
this edits the profile status
yes
but what does rpc have to do with editing the profile status?
is either an option I missed or
not really selfbot in my opinion as it requires you to actually do something (start some registered application)
but maybe spammy no clue
my fault ill rename it rn
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
oh
git diff
why can you forward embeds individually
Commit 8fa7d00 breaks compiling on windows anymore
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
this is the only broken thing I can find between Vencord Vesktop and the Installer
anyways im off 2 work
Discord Account
CJStylesOrg
What happens when the bug or crash occurs?
When I delete messages starting from the top of the thread downwards, the messages automatically collapse again after each deletion from the top. However, if I delete messages from any other position in the messages (not starting from the top), the collapsed messages remain expanded and do not collapse.

What is the expected behaviour?
I expect the message lo...
i wouldn't really consider this an issue, seems like expected behaviour
blocked messages do the same too
@brazen bone what do u think?
oh wait while we're at the topic of collapse messages now, i notice that when i turn off the option and delete a message, it continues to collapse newly deleted messages until you relaunch discord.
assuming their was supposed to be a prompt telling you to relaunch discord, same issue i made about showhiddenthings which was fixed.
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
Smells like (very minor) discord bug to me
it is yeah
yet another "very minor discord bug" exposed by a vencord plugin
i can relate 
if its only exposed by a plugin, is it really a bug
neither the bug mentioned here and the bug I found yesterday are only exposed by a plugin
both can be found in stock discord, albeit slightly rarer to experience
ah
Have blocked user post two messages, uncollapse, delete first
Bug I found: Open mod view from context menu, no highest role because discord stupid and goes through membersafetystore to get that
Probably (without looking at the code) the collapsed message group is keyed by first message id, so when the first id in the group changes, the useState is reset
No idea how you'd go about fixing that
Commit 8fa7d00 breaks compiling on windows
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:' code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
This might be caused by the actual file extensions using mjs still instead...
My helpful assistant @glass jasper has tested this for me and this has fixed it
I was apparently too tired to even notice it was still mjs until now ig
idk what happened he just blew my computer up telling me what to do.
You are now infected hahahhaha
pretty sure that started when you got me on board helping with Equicord (/joke)
mjs is correct tho
that's deliberate
esm is a fuck
I dont even know anymore
it wont compile for me
but compiles for indi
.mjs is right
so
why
gee
its adding a third /?
the asar stuff
I cant build after the esnext updates
wdym esnext lol
idk just what I remembered from the name
of the commit
that commit
sorry esbuild
its not that easy to fix
but outside of that building works fine
tf
what command are you running
ye just merged it
try on node 20
added support for channel mentioning and polls
this has been implemented via 3ce2410
Thanks for adding this! It fits well into the Plugins menu and makes it much easier to review code and find issues! (Don't want anyone stealing my cookies e-e)
I guess you had the idea already before I asked about it here, but its all done now and should help some other nerdy users too.
added support for voice messages, keep in mind that i tried my best to ensure the filter values are as identical as the status danger variable, feel free to change it before this gets merged in the future to the best liking.
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
of course they are not gonna merge canary into stable for the whole weekend
Why is friendssince so fragile?
assuming discord is really actively working on profiles, since canary got a new look with the new add friend and message button.
previously it was behind an experiment.
Discord LOVES redesigning the user profile in the DM sidebar
i think that is why it added duplicate @ symbols in the mod view in my snippet.
screenshot?
mod view internal code is disgusting tbh

this is what it looked like days ago, i just grabbed the old code before i made changes to it.
simple fix though
fun fact
you cant see this highest role in mod view on anyone here
Try it.
The role just doesn't appear, at all
oh yeah im the one that made the issue and you responded to it.
oh 
that is honestly stupid though, im sure this new refresh their doing is them trying to sort out this mess.
how tf did I not even realise that
doubt
When was the last time discord sorted out a mess
idk 
mod view needs other fixes too
why doesn't it show what time today something happened
knowing "It happened... today" is entirely useless
not to mention how it doesn't show who the action was taken against, only who took the action (sometimes?)
at first i thought the @ symbol was from my snippet but realized it was there before-hand.
was this added by the plugin or discord?
what plugin are you talking about
the @ symbol in the mod view action logs.
@rugged spire, <t:1721340367:R>: revive GifRoulette and make it even more obnoxious
mood
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
aaa not the debug stuff
Such good commit messages
mapMangledModule such an advanced find
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
I caused this!!!
what did you do
f9fb3bb fix watch erroring when clean building - Vendicated
volume button path fix and voice message waveform more identical to the deleted color.
okay so if you update the asar but don't restart the app and only soft reload, it uses state of the old asar which causes this shit
(fixed by full restarting)
so we should force a full restart
or is there some way to invalidate cache of asar
happens on BD too xd
i think vencord should force restart after an update
actually no we cant
cause of auto update
how does discord solve this in their updater
maybe download update to vencord.asar.new and apply on before-quit?
π
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
it doesnt work...
π
i think this is the only option
which means only actually applying the update after a full restart regardless right
yes
if I use the inject script will it use the old installer which doesnt support asar
okay yes it will
yes
you need to build new installer
or
its probably easier for me to manually install than to clone and build windows
u can do it really easily
go into patcher.js, search for Vendicated/Vencord and change it to Vendicated/VencordStaging
then update via in app updater
well that's for standalone xd
isnt it easier for me to manually inject
kinda
should I look into the code for the installer or its quick to explain what to do
well what do u want to test exactly
anything lmao
first of all, check out the branch and build and ur dev install should still work without having to reinstall
other than that just build the installer from source and put it where the runInstaller.mjs script downloads installer to
if you build the cli instead of gui, you only need go installed and nothing else and it will compile fast
go build --tags cli
that'll build the cli
yeah but this is not a proper install
it just overwrites dist/patcher.js with that code
because the installer puts app.asar which is hardcoded to import AppData/Vencord/dist/patcher.js
so users would have to reinstall
I see
but yeah just build the cli it won't take u long
building gui neds a bunch of c++ libraries and 5+ minutes to compile so don't do that xd

anyway i would say the PR is pretty finalised now
just need to do a bit more testing and then we can release it
hopefully tomorrow
yes
nothing changed about it
it's the same as before, just slightly different app.asar contents
alright it worked
btw the new installer should also just work when run in normal mode cause i uploaded a testing build to main repo releases
okay how do you cause this explosion?
change anything and build again or something in particular
you need to change vencord.asar to a different build and then reload
so you need to do a standalone build?
but anyway this should fix it
i wouldn't bother looking more into fixing it
the apply on quit solution is perfect imo
it will also prevent issues with making change in main and then people only reload renderer and renderer expects new main but really has old main
happened a few times in the past
alright
the way i was testing updater is that i just made a separate repo at https://github.com/Vendicated/VencordStaging
and pushed random commits
who tf starring that repo
we can delete all .js, .map and .LEGAL.txt files except patcher.js
why is patcher needed still?
if u wanna help me test updater:
- make a repo like this ^
- create devbuild release (empty) and enable workflows
- push my asar branch to main branch in ur repo and make sure that it creates release
- install normal vencord with the old official installer, open
patcher.js, and change allVendicated/Vencordto your repo you made - update once with in app updater
- now when you open Discord next time, it should migrate you to new asar install
- now push some commit to your repo and update with in app updater
I will help you test rn since after tomorrow I will be far away from my pc
for days
scary
so this is the last time I can lol
just make sure that it works
and uhhh be very careful to push to the right repo and not vencord main repo

@austere talon do I need to add any env vars?
oh rightt
@austere talon I updated and choose to not restart
then when I clicked to exit discord using the tray
which one
vencord.asar
okay
yeah cause it applies old updates on boot
yep
anyway it's not hard to fix
just need to
- rename
vencord.asar->vencord.asar.old - rename
vencord.asar.new->vencord.asar
on boot:
- delete
vencord.asar.old
maybe u wanna do that
if not thats okay, i'll do tomorrow
I dont think that will work
unless using the explorer is different than using the apis
self_updater.go: Lines 113-121
if err = os.Remove(ownExePath); err != nil {
if err = os.Rename(ownExePath, ownExePath+".old"); err != nil {
return fmt.Errorf("Failed to remove/rename own executable: %w", err)
}
}
if err = os.Rename(tmp.Name(), ownExePath); err != nil {
return fmt.Errorf("Failed to replace self with updated executable. Please manually redownload the installer: %w", err)
}
try it with pwsh / cmd
did it work
nop
^^
well i'll try tmrw
dumb suggestion
why don't you just perform this logic before vencord.asar is even loaded using a stub loader of some kind
alright
hi! this plugin will always ping the user when they are replied to, regardless of the ping toggle
please provide your feedback and suggestions!
Looks cool actually, the description could use some work
i think the name could be a tad more descriptive
That sounds like it would ignore the toggle on send, not receive
Yeah that's what i was thinking
Wait, sorry if I'm wrong, but, doesn't #1975 supersede this?
Wait, sorry if I'm wrong, but, doesn't #1975 supersede this?
From what i can tell, that plugin modifies the actual message component to look like it mentioned you (i might be wrong)
https://docs.vencord.dev/installing/
Docs seem to be wrong on building from src, use buildWeb instead of build --web on windows, looking @ package.json and the scripts with gitblame I don't see when/how this changed recently.
#π₯-vencord-support-π₯ message is there a better channel for this?
buildWeb was always a seperate script, whoever put that in the docs was wrong
Looking back further, it was always wrong, heh.
β¦o to crash your client when viewing the profile
Example: putting this: "σ σ £σ ¦σ ¦σ ¦σ ¦σ ¦σ ¦σ ¦σ ¦σ ¦σ ¬σ £σ ₯σ ₯σ " in your bio will make anyone using the plugins client crash when viewing it. Decoding it looks like "[#fffffffff,#ee]". It crashes because it doesn't validate the hex codes to be only 6 chars long, which then crashes the client.
It would be ideal to allow 3 length as well and use tinycolor (shipped with the client) to properly convert it to an integer.
Just don't fault me for originally writing this naively when it wasn't ever meant to get this far anyways.
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
None
None
None
None
I've been doing this for a while now and I can confirm it works
The copy 3y3 button always gives a 6 char hex code, so I don't necessarily see the need of including support for it.
Anyone know how to deal with scrollers? Currently when a message is very long the ML modal has a scrollbar over the entire contents including the tab bar, but I'd rather have the tab bar stay at the top and only scroll the message content
src/webpack/common/types/components.d.ts: Lines 457-468
export type ScrollerThin = ComponentType<PropsWithChildren<{
className?: string;
style?: CSSProperties;
dir?: "ltr";
orientation?: "horizontal" | "vertical";
paddingFix?: boolean;
fade?: boolean;
onClose?(): void;
onScroll?(): void;
}>>;
Looks like it
Doesn't seem to work, probably because it's already inside another scroller
oops, wrong way.
meant to merge vendicated/vencord into my branch - mb lol
oops :p
Discord Account
No response
What happens when the bug or crash occurs?
Every action is saying twice.
I can send an example if u need
What is the expected behaviour?
The narrator only speak once
How do you recreate this bug or crash?
Its happening everytime (even after restart app)
Errors
No error
Request Agreement
- [X] I am using Discord Stable or tried on Stable and this bug happens there as well
- [X] I have read the requirements for opening an issu...
can't reproduce. use our support channel if you need help.
Comparison chart of SignPath editions
Oh I didnβt scroll down
it would unfortunately still require us to leak one of our names or establish a company (which will still require that anyway)
- globalsign certificates need to belong to a legally registered org
i dont know if signpath take that responsibility when issuing for open source
ah
it comes up as signpath foundation
i guess thats not a bad solution
we'll see
we have to put it through their infra though which is kind of a blackbox

Discord Account
hauntii
What happens when the bug or crash occurs?
VcNarrator will say all actions that occur twice if Discord Overlay is injected into a game.
What is the expected behaviour?
VcNarrator will only speak the actions from the main Discord window.
How do you recreate this bug or crash?
- Enable VcNarrator
- Open a game with Discord Overlay enabled
- Observe it speaking twice
Errors
None
Request Agreement
- [X] I am using Discord Stable or t...
it is, youre right
switch to alt
create new issue
disregard comments from vee
if its been reported twice its prob an actual issue, i can't replicate though
wrong, the commit number is 686f2d9 
oh true, the extra f
i love how that person just scrolls all the way down the commits just to comment on a random commit from december of 2023.
posts useless comment
comment is wrong
If their infra is ever compromised, there are way bigger issues then some APT spoofing a vencord install.
And I don't think 99% of companies should be self hosting code signing.
It's hard, plus the dongles.
Tons of benefits for very little risk
wrong, extra f added
@Vendicated is this too niche in your opinion?
literally every company that buys their own certificates either goes through a cloud blackbox or, if required for auditing purposes (so most companies), uses the dongles/provided server
you can get a signing server to install and put on the network
blackboxes and blind trust are not good security practice (see: literally crowdstrike)
i'd rather not sign if there's no way for us to actually sign our code ourselves
i dont trust random companies that offer codesigning for free if you dont get the certificate
plus most companies dont want to submit their code through the cloud blackbox because of trade secret reasons
im aware this is different because its OSS software, but we still dont know what they're doing and they don't publish the process anywhere
i give it an executable, i get a different, signed executable back out
i dont know whats going on in between
i wish we could get the old open source developer certificates back
those were fire
and you got to actually keep them yourself and run your own signing process
@rugged spire, <t:1721513189:R>: revive GifRoulette and make it even more obnoxious
you know what i have been pushing this back too much no i am not fixing
That's the issue.
(From the stance of prodsec of a security vendor) Securing the network and having a prodsec team to analysis stuff when it gets flagged (supplychain attack style) is not easy. SLSA is going to help get companies to where they need to be but OSS does not have the infra in place for it.
I don't see most of the above from my view of things, but maybe it's different in SMB?
https://azure.microsoft.com/en-us/products/trusted-signing would be an alternative for companies, but requirements are a lot higher.
Verifying code is super easy now too. Heck, Ghidra is free and you can do so much with just hashes.
I get the FOSS communities want to DIY stuff, but somethings are better to be secured at scale where more resources can be spent
Secure your applications with a fully managed end-to-end signing service for code, documents, applications, and more with Trusted Signing from Microsoft Azure.
And that's why most companies have horrid security, then try to selfhost and then their cert is leaked for cheatdevs. Cloud based solutions are better then what 99% of companies can DIY. ;-;
i wouldnβt want to run an exe from a black box
Unless you are building from source, GHA is a black box π¦ (SLSA + syscall tracing would help tho)
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
PictureInPicture (had no effect):
ID: 546432
Match: ```
/.nonMediaMosaicItem]:!(\i).{0,10}children:[(\S)/
**__FriendsSince (had no effect):__**
ID: `926150`
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i\.id)}\)}\)/
None
None
None
friendssince my beloved
what does discord have against friendssince?
they're messin around with profiles
oh they finally pushed it to stable
most likely getting ready to probably change the message box in profile popouts to just a message button.
this is using the cta message and call experiment on canary but they removed an experiment recently which changed the design of the add friend button, forcing it to the stable build too, this is likely next.
FriendsSince (had no effect):
ID: 926150
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i.id)})})/
None
None
None
PictureInPicture (had no effect):
ID: 546432
Match: ```
/.nonMediaMosaicItem]:!(\i).{0,10}children:[(\S)/
**__FriendsSince (had no effect):__**
ID: `926150`
Match: ```
/BOT_PROFILE_CREATED_ON,.{0,100}userId:(\i\.id)}\)}\)/
None
None
None
huh
@limber skiff u love
they broke again xdddd
looks like old patch works again
None
None
None
None
PictureInPicture (had no effect):
ID: 546432
Match: ```
/.nonMediaMosaicItem]:!(\i).{0,10}children:[(\S)/
None
None
None
None
None
None
None
None
None
None
None
lool
RelationshipPruner allows you to easily go through and remove your friends/servers (useful, especially if you have a lot). You can view relevant information to help you make the decision
There may be some parts that could be improved, mainly the api requests in the guild prune modal- they should be fine but it's...
I forgot to mention, right now it has an issue with the mutual servers/last message date, it only loads if you have loaded the user in your session (i'll fix soon)
why c# style
You saw nothing
missed a few spots lol
is that specifically a c# thing? thought it was just how some IDEs formatted code by default
most c# code i see or write looks like that
also ides tend to format it like that by default
horror
C# WAS MY FIRST LANGUAGE LEAVE ME ALONE π
I'll fix soon
REAL
it's conventions are disgusting though
-a c# dev
it is conventions are disgusting though
You need help
auto correct, vee, auto correct..
@austere talon hello sorry for being months late never noticed you responded to my PR cause i dont check my email often
you suggested this change however when i remove the count thing and there are more users than the max (12) the count is needed for it to show "+1" or +2 wtv properly
with
without
fuck i broke something
im gonna cry i dont understand git
is there any chance i could close my old pr and make a new one because idk how to fix git issue
what's the issue
i didnt touch the pictureInPicture thing at all
and some stuff are red in constants when i just added myself
I will resolve for you
actually no I cant
what's the name of the vencord repo remote you have
upstream? origin?
alright i fixed it i think i just uh
went to github.com/Vendicated/Vencord and copypasted the files it says i modified into my local repo
I got rid of the guildId and showUserPopout ones as they weren't needed, renderUser is necessary in order for the avatar to be clickable and renderMoreUsers is necessary in order to show a custom tooltip
Let me know if I can resolve this conversation
I believe this line is needed in order to properly display the more users thing
With
Without
Doesn't it make it worse?
13 is definitely not many people + 12
sorry for my wording: "+12" is wrong and thats what it says without the line I was suggested to remove, with the line it shows "+2" which is accurate
Oh, my bad
I thought with/without this change
i hate this formatting so much
so many wasted lines for no reason
oki doki i think the mod is done, and ready for submission!
oops
insightful
i didnt realise they changed it already
pointing stuff out on drafts isnt the end of the world lol
not enough arrows, i dont know what im suppose to be looking at
it just-so-happens that they already caught it
I'd rename this to connectionTypes
Is there a reason to not just put the function here? rather than calling it?
github webhooks how i dont love you <3
You will push 0 commits
i would heart emoji react that but im kinda in a weird state of schrodinger's programmer which prevents me from doing so
because I didn't sync anything yet (this was a force push commit from vee)
gotta love force pushes. π€¦
thanks for the heads up whoever you are!
hm?
ty
Description
Create option for link hiding via ghost messages (client parser bug that has been around for a while with evidently no action from discord to fix it). Adding for feature parity with the BD plugin.
It's off by default because it doesn't work particularly well for multiple emotes and editing (you have to delete the links yourself), although it will append new emotes upon edit.
This more for people like me who are bothered by the little changes when switching over from BD; ...
cant wait for ppl to complain abt it not working/spam cause a nonzero amount of servers have the spoiler thing automodded
100% any normal auto will block this
This affects that red button in the plugin settings, that errors out instead of switching to dark theme:
I searched for this here but what about switching to bun for faster deps install times? Bun is already for windows and it should work fine, i can also make pr but asking first. Well yeah it's maybe pointless as vendicated said earlier but still it reduces time.
DeepL API has a CORS policy that blocks requests from the frontend. A way to deal with this would be to use a proxy server, but then you would be sending your API key and potentially private message content to an individual's proxy server before DeepL.
DeepL API has a CORS policy that blocks requests from the frontend. A way to deal with this would be to use a proxy server, but then you would be sending your API key and potentially private message content to an individual's proxy server before DeepL.
we can just delete the CORS header in desktop clients (electron based) in the same way that CSP is modified https://github.com/Vendicated/Vencord/blob/main/src/main/index.ts#L90-L124
we can just delete the CORS header in desktop clients (electron based) in the same way that CSP is modified https://github.com/Vendicated/Vencord/blob/main/src/main/index.ts#L90-L124|
it's also possible to make a "native" script that fetches that data for you
has this been done by an existing plugin yet?
we can just delete the CORS header in desktop clients (electron based) in the same way that CSP is modified https://github.com/Vendicated/Vencord/blob/main/src/main/index.ts#L90-L124|
it's also possible to make a "native" script that fetches that data for youhas this been done by an existing plugin yet?
https://github.com/Vendicated/Vencord/blob/main/src/plugins/openInApp/native.ts
I'd like to re-open this issue as this only happens after injecting Vencord into the stock client with a version above 0.0.59.
The error doesn't show when Vencord is not injected and can be reproduced 100% of the time.
I wonder if it could be linked to the NoTrack plugin in some way
https://github.com/Vendicated/Vencord/blob/main/src/plugins/openInApp/native.ts
cool, I actually got everything working using a native script. however, this means that only google translate would be available on web, and deepl would only work on the desktop client, right? would this just make the entire translate plugin incompatible with web, or would there have to be some sort of built-in check for the platform before letting the user switch to deepl?
why
pnpm is already very fast
and it's also irrelevant
I've always had faster installation times with bun and also running scripts from pkgjson but yeah if it's irrelevant then there's no reason to do it
Building is already just about instant, especially with pnpm watch
Also bun is still very new
pnpm and esbuild πͺ
also installing a whole new runtime just for package management seems a bit pointless

Adds an option to the Translate plugin to use DeepL Free or Pro with an API key.
- Addresses feature request #1786.
- Google Translate remains the default translation service.
isn't this not allowed?
i got explicit permission from ven #π§©-plugin-development
makes sense
lol
like the whole part? or a certain part?
that was put their before the read me and is basically the things on why i can't do certain things and stuff
the part i commented on
ik i said in le thing
no, i mean what does the line i commented on mean
what line, u commented on four lines
no i didnt
well it shows that so um was it the facebook line
mhm
k
I can't get user profile ids because it shows the user id that apps are given which is different from ur actual user id. I also don't have access to discord's token they use for authorizing with facebook so I can't get the actual profile.
thats very cool π
vouch
I think directly using restapi is a bad idea
Discord Account
@lncvrt
What happens when the bug or crash occurs?
The bug is the icon for platform indicators console is messed up
What is the expected behaviour?
Should be a proper icon
How do you recreate this bug or crash?
Find a person on a console then use platform indicators
Errors
No response
Request Agreement
- [X] I am using Discord Stable or tried on Stabl...
No, fridge sounds about right, speaking as a pc gamer
Since when does discord run on consoles
xbox and ps5 have it now
I think this plugin is ready for final review @Vendicated and @Nuckyz!
I believe that image is correct, it's showing a fridge.
Well it's a Xbox Series X from the angle like the picture below
I mean the icon looks like its more like rotated 35Β° to the left but whatever.
it doesn't really make much sense to me though
theres a lot better icons like a xbox one for example
ARE YOU FUCKING KIDDING ME, RENAMING A BRANCH DOES THAT? WTF
I love github
said no one ever
someone find for me the name of the store that has the getLocalPresence function
selfpresencestore
EditMessageStore
thank youu
just ooc, how are you finding these
I'm cleaning up my branch without touching discord code since I'm using a potato laptop
findByProps(funcname) -> show function definition -> scroll down a bit to find the displayname
lemme make a cid
vid
For me it's just a quick grep
another one
getMessageRequestsCount
UploadAttachmentStore
nice
Easy stuff
scary
allSummaries, findSummary
SummaryStore
getPendingReply
@limber skiff just ooc, whatre ya doin?
also is getPlayerState from a store (Spotify related)
cleaning up a branch and also converting findByProps to findStore when possible
PendingReplyStore
yop, SpotifyStore
SortedVoiceStateStore
does it have getCurrentClientVoiceChannelId too?
I see
π§Ή
no.
ImageZoom (errored):
ID: 124347
Match: ```
/placeholderVersion:\i,/
Error: ```
Invalid destructuring assignment target
SpotifyControls (found no module):
ID: -
Match: ```
"AccountConnected"
findByCode("type:"UNSYNCED_USER_SETTINGS_UPDATE",settings:{useSystemTheme:"system"===")
None
None
oh so it wasn't all my fault
It has to be {0,6} to maintain compatibility with similar plugins for other client mods.
/\u{e005b}\u{e0023}([\u{e0061}-\u{e0066}\u{e0041}-\u{e0046}\u{e0030}-\u{e0039}]{0,6})\u{e002c}\u{e0023}([\u{e0061}-\u{e0066}\u{e0041}-\u{e0046}\u{e0030}-\u{e0039}]{0,6})\u{e005d}/u,
insane bug
crazier regex
though i'm pretty sure that regex already was huge
It's only crazy because of all the \u{}, without that it looks pretty tame
It's just \[#([a-fA-F0-9]{0,6}),#([a-fA-F0-9]{0,6})\], but with tag characters instead of ascii
I know :(
Absolute madlad
What are those fixes tf
Did discord actually name something isCopiedStreakGodlike lol
I just matched it to "account";
this thing
What the
I'm confused on why the look back
wasnt it originally the thing when u click on ur profile here it copies the username, but copying it multiple times gives u the easter egg
Yes
because other wise it adds to let {foo: bar} = e;
If I'm not mistaken discord just added the let correct?
Or are we looking 2 different places
It seems to work fine with the match match: /dataSafeSrc:\i,children:/, if you want you can test it
but why should i change it again
I don't think the look back is necessary
Lookback kabooms your performance too iirc
That's all I'm saying
Don't count me on that
One millisecond on load doesn't really matter
it doesnt much here
- Words of someone not using a laptop
when was your laptop made 
Unless your laptop is moving at relativistic speeds, a millisecond is equally long on all hardware
If your laptop is moving at relativistic speeds, I lift my hat to you
before it was ,placeholderVersion:foo}=this.props; but they added dataSafeSrc after that so now it matches that instead
But the current code doesn't support non 6 char hex codes. #fff will not be interpreted as white by the current code, but rather as #000fff, which is blue. Therefore I don't see the need for allowing shorter codes
Your matching the wrong thing then?
If it ended at : foo } then it was matching below it because of the ,
no the current version matches the wrong one
I'm so confused
If your matching the thing below the let then why the look back it makes no sense
You can just match to what's after placeholderVersion and it's fine
I thought you were patching the let mb
... because it regex doesn't know it below the let? Thats what im doing with the lookbehind
It does tho?
I did it?
Don't match placeholderVersion?
Match dataSafeSrc?
Wait
I'm so lost
what if they add more variables to destructure
we will back here again
.{1,50} is what I used to get around that
At this point just match children
,children: (joke)
apparently you can now use listening RPC on canary
so the activity fixer is no longer needed
ImageZoom (errored):
ID: 124347
Match: ```
/placeholderVersion:\i,/
Error: ```
Invalid destructuring assignment target
SpotifyControls (found no module):
ID: -
Match: ```
"AccountConnected"
findByCode("type:"UNSYNCED_USER_SETTINGS_UPDATE",settings:{useSystemTheme:"system"===")
None
None
el patch.
oh broken on stable too
ImageZoom (errored):
ID: 124347
Match: ```
/placeholderVersion:\i,/
Error: ```
Invalid destructuring assignment target
SpotifyControls (found no module):
ID: -
Match: ```
"AccountConnected"
findByCode("type:"UNSYNCED_USER_SETTINGS_UPDATE",settings:{useSystemTheme:"system"===")
None
None
just a reminder this PR exists
unsure of it it is a Good Fix or not
for the ClientTheme thing (the bad find), I did that a bit ago https://github.com/Vendicated/Vencord/pull/2720
None
None
None
None
None
None
None
None



