feat!: assistive-pricing as a Worker

[gentlementlegen]

Resolves #5

[github-actions[bot]]

Unused devDependencies (5)

Filename	devDependencies
package.json	@types/jest esbuild eslint-config-prettier eslint-plugin-prettier ts-node

Unlisted dependencies (1)

Filename	unlisted
tests/main.test.ts	@jest/globals

Unused exports (4)

Filename	exports
src/handlers/pricing-label.ts	labelExists
src/shared/label.ts	deleteLabel
src/shared/pricing.ts	calculateDurations
src/types/typeguards.ts	isIssueEvent

Unused types (8)

Filename	types
src/types/env.ts	Env
src/types/github.ts	Comment Repository WebhookEvent
src/adapters/supabase/types/database.ts	Tables TablesInsert TablesUpdate Enums

Unused Enum Members (2)

Filename	Enum	Member
src/types/github.ts	UserType	User Organization

[github-actions[bot]]

Lines	Statements	Branches	Functions
	83.33% (20/24)	66.66% (6/9)	80% (4/5)

JUnit

Tests	Skipped	Failures	Errors	Time
2	0 :zzz:	0 :x:	0 :fire:	3.925s :stopwatch:

Coverage Report (83%)

File	% Stmts	% Branch	% Funcs	% Lines	Uncovered Line #s
All files	83.33	66.66	80	83.33
src/handlers	84.21	66.66	100	84.21
command-parser.ts	84.21	66.66	100	84.21	16, 19, 27
static	80	100	66.66	80
main.ts	80	100	66.66	80	9

[gentlementlegen]

Major changes

• assistive-pricing now running as a Worker
• fixed Knip and reported errors
• related tests and Jest running at pull_request
• fixed command error being displayed all the time even for unrelated commands

[gitcoindev]

Hi @gentlementlegen there is a small commit needed for knip workflow that removes superflous steps. See comment from @rndquu https://github.com/ubiquity/rpc-handler/pull/13#issuecomment-2123006059 . This does not touch any of your changes, I will add this commit to the pull request.

[gitcoindev]

Hi @gentlementlegen there is a small commit needed for knip workflow that removes superflous steps. See comment from @rndquu ubiquity/rpc-handler#13 (comment) . This does not touch any of your changes, I will add this commit to the pull request.

My bad - I checked and you pulled latest changes already, thank you! I also checked on a local workspace

✂️ Excellent, Knip found no issues.

[gitcoindev]

@gentlementlegen I checked out the pr workspace, and I am able to test locally with jest. Would you mind adding to README.md steps needed to execute and test the worker locally including supabase setup, if needed? I set supabase .dev.vars to my instance but I see {"error":"Error: null is not a valid content type"} when I connect to worker started with yarn worker command and I am not sure what I am missing and what to expect. Perhaps adding this info would make life easier to anyone who would work on the plugin afterwards. I also see cypress is still a placeholder, will it be used in this repository?

[gentlementlegen]

@gitcoindev Thanks for the feedback. I removed Cypress, and added instructions in the README plus a .http test file. Let me know if that is clear enough or if I should add more instructions.

[gitcoindev]

@gentlementlegen thank you for adding the example, I am approving the pull request from my side.