Merge development into main

[gentlementlegen]

No description provided.

[github-actions]

Lines	Statements	Branches	Functions
	36.9% (186/504)	10.93% (21/192)	32.14% (27/84)

JUnit

Tests	Skipped	Failures	Errors	Time
4	0 💤	0 ❌	0 🔥	5.399s ⏱️

Coverage Report (40%)

File	% Stmts	% Branch	% Funcs	% Lines	Uncovered Line #s
All files	36.9	10.93	32.14	40.26
src	73.01	33.33	45.45	72.13
run.ts	59.25	20	16.66	59.25	23–35, 46–51, 56
worker.ts	83.33	50	80	82.35	17, 36, 45–53
src/adapters	100	100	100	100
index.ts	100	100	100	100
src/adapters/supabase/helpers/tables	39.02	0	54.54	44.44
access.ts	27.77	0	50	31.25	14–26, 31, 43–50
label.ts	20	0	25	25	27–54
super.ts	100	100	100	100
user.ts	100	100	100	100
src/handlers	31.33	10.63	21.95	34.11
command-parser.ts	79.31	60	71.42	79.31	22, 29, 32, 40, 65, 71
comment.ts	80	36.36	100	82.05	10, 20–23, 30, 38, 73
handle-parent-issue.ts	35.71	0	0	41.66	7–11, 15, 19–20
label-change.ts	9.37	0	0	9.67	6–57
pricing-label.ts	7.21	0	0	8.23	12–167
sync-labels-to-config.ts	14.28	0	0	16.66	9–33
src/shared	21.37	7.69	25	25
issue.ts	77.27	54.54	100	89.47	21, 51
label.ts	15.38	0	0	16.66	9–22, 26–28, 37–56, 62–75, 80–93
permissions.ts	11.62	0	0	12.19	8–62
pricing.ts	7.31	0	0	10.71	5–7, 11–29, 33–41
src/types	87.5	50	50	87.5
env.ts	100	100	100	100
github.ts	100	100	100	100
plugin-input.ts	100	100	100	100
typeguards.ts	66.66	0	33.33	66.66	15, 24

[gentlementlegen]

@0x4007 Required changes are within #11

[0x4007]

@0x4007 Required changes are within #11

Why didn't you include them in the same pull? I could have just viewed the diff from my last review to easily confirm that everything was addressed here.

Also knip looks broken now.

[gentlementlegen]

@0x4007 I cannot edit this PR as it is within this repo (not from a fork where I have permissions) so I have to add another PR (or edit with VSCode within the browser like we tried yesterday).
The broken Knip is with pull_request_target which should not exist, and shall stop running in favor of pull_request one which is green.

[0x4007]

@0x4007 I cannot edit this PR as it is within this repo

None of these observations make sense (in reference to you not being able to edit my pull the other day.) You opened this pull so regardless of where the branch is located you have write permissions. You're not doing something right.

[gentlementlegen]

@0x4007 Could be because I always use https login and not tokens, or because I login through Jetbrains OAuth probably, I will figure this out. In the meantime, changes have been merged here, so lemme know if this can be merged too.

[0x4007]

After you address my review I think you should just merge.

[gentlementlegen]

@0x4007 I will replace messages and fix error codes to be 4XX as per the HTTP RFC. 5XX should never occur except on crash on unhandled errors by the server (aka the Action).

A great description on how to chose error codes: https://stackoverflow.com/a/39639949/10346405