ubiquity-os / ubiquity-os-plugin-installer

A GUI to install UbiquityOS plugins.
0 stars 8 forks source link

Create add/remove config logic #3

Closed 0x4007 closed 1 day ago

0x4007 commented 2 months ago
  1. We need to be able to select which ubiquibot-config.yml to edit, so we at minimum need an organization picker after GitHub authentication.
  2. We need to parse the YML file, and target the config based on the plugin's URL/location. Then edit the YML file and push the commit using the user's credentials.

More thoughts on the readme.

LLM

Without research, the lazy answer is that LLMs can easily solve the problem. We could even consider passing in the most important/relevant files of the plugin so that the LLM can figure out all the default values and other miscellaneous details.

But it would certainly be preferred to implement this with traditional code. I think with a bit of research it can be pretty straightforward.

Neuman-Nz commented 1 month ago

/help

ubiquity-os-beta[bot] commented 1 month ago

Available Commands

Command Description Example
/help List all available commands. /help
/allow Allows the user to modify the given label. /allow @user1 label
/query Returns the user's wallet, access, and multiplier information. /query @ubiquibot
/start Assign yourself to the issue. /start
/stop Unassign yourself from the issue. /stop
/wallet Register your wallet address for payments. /wallet ubq.eth
ubiquity-os[bot] commented 1 month ago

Available Commands

Command Description Example
/help List all available commands. /help
/allow Allows the user to modify the given label. /allow @user1 label
/query Returns the user's wallet, access, and multiplier information. /query @ubiquibot
/start Assign yourself to the issue. /start
/stop Unassign yourself from the issue. /stop
/wallet Register your wallet address for payments. /wallet ubq.eth
Neuman-Nz commented 1 month ago

/start

ubiquity-os[bot] commented 1 month ago
! Please set your wallet address with the /wallet command first and try again.
ubiquity-os[bot] commented 1 month ago
! No wallet address found
0x4007 commented 1 month ago

@gentlementlegen can you fix the double post?

gentlementlegen commented 1 month ago

@0x4007 Spec at https://github.com/ubiquity-os-marketplace/command-start-stop/issues/60

Apetree100122 commented 1 month ago

/start

ubiquity-os-beta[bot] commented 1 month ago
Deadline Tue, Oct 8, 5:33 AM UTC
Beneficiary 0xC6DcC5424AE41dec50e9444742f83Cd489D5527a

[!TIP]

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.
ubiquity-os[bot] commented 1 month ago
Deadline Tue, Oct 8, 5:33 AM UTC
Beneficiary 0xC6DcC5424AE41dec50e9444742f83Cd489D5527a

[!TIP]

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.
Keyrxng commented 2 weeks ago

starting this now and I'll just link the PR again, it'll be building on top of #12.

ubiquity-os-beta[bot] commented 1 week ago
Deadline Sun, Nov 10, 6:01 AM UTC
Beneficiary 0xAe5D1F192013db889b1e2115A370aB133f359765

[!TIP]

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.
gentlementlegen commented 1 week ago

Funny, I had made a tool for my personal usage a while ago, I didn't remember this task.

Keyrxng commented 1 week ago

Funny, I had made a tool for my personal usage a while ago, I didn't remember this task.

If it's better than my approach pls share and I'll use it here

gentlementlegen commented 1 week ago

@Keyrxng It is a very different approach! At first it was just for editing the configuration files in situ. Also I used NextJs and Material which I don't think is welcome haha.

ubiquity-os-beta[bot] commented 1 day ago
+ Evaluating results. Please wait...
ubiquity-os-beta[bot] commented 1 day ago

 [ 600.6225 WXDAI ] 

@Keyrxng
Contributions Overview
ViewContributionCountReward
IssueTask1600
IssueComment20.6225
ReviewComment280
Conversation Incentives
CommentFormattingRelevancePriorityReward
starting this now and I'll just link the PR again, it'll be buil…
1.22
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 19
  wordValue: 0.1
  result: 1.22
0.530.4725
If it's better than my approach pls share and I'll use it here
1
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 15
  wordValue: 0.1
  result: 1
0.230.15
Resolves https://github.com/ubiquity-os/ubiquity-os-plugin-insta…
4.5
content:
  content:
    p:
      score: 0
      elementCount: 16
    ul:
      score: 0
      elementCount: 2
    li:
      score: 0.5
      elementCount: 9
    hr:
      score: 0
      elementCount: 1
  result: 4.5
regex:
  wordCount: 281
  wordValue: 0
  result: 0
0.830
@0x4007 saw me specify `service_key` and said that we sh…
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 24
  wordValue: 0
  result: 0
0.630
Should be taken care of within the template directly more import…
0
content:
  content:
    p:
      score: 0
      elementCount: 2
  result: 0
regex:
  wordCount: 23
  wordValue: 0
  result: 0
0.430
template issue
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 2
  wordValue: 0
  result: 0
0.330
![image](https://github.com/user-attachments/assets/b213fd68-831…
5
content:
  content:
    p:
      score: 0
      elementCount: 1
    img:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 0
  wordValue: 0
  result: 0
0.130
No that's `list all organizations` I'm calling `list…
0
content:
  content:
    p:
      score: 0
      elementCount: 3
    hr:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 32
  wordValue: 0
  result: 0
0.530
It's likely failing due to your permissions.When I was showing…
2
content:
  content:
    p:
      score: 0
      elementCount: 6
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 4
  result: 2
regex:
  wordCount: 79
  wordValue: 0
  result: 0
0.630
It was failing for some reason so I just removed it as it was a …
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 26
  wordValue: 0
  result: 0
0.430
1. `manifest.name` should reflect the repository name so…
2
content:
  content:
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 4
    p:
      score: 0
      elementCount: 7
    hr:
      score: 0
      elementCount: 1
  result: 2
regex:
  wordCount: 97
  wordValue: 0
  result: 0
0.730
@gentlementlegen, is null valid for `text-conversation-rewar…
10
content:
  content:
    p:
      score: 0
      elementCount: 3
    img:
      score: 5
      elementCount: 2
  result: 10
regex:
  wordCount: 36
  wordValue: 0
  result: 0
0.530
Understood.That's the kinda thing I'm trying to think how can …
0
content:
  content:
    p:
      score: 0
      elementCount: 5
  result: 0
regex:
  wordCount: 69
  wordValue: 0
  result: 0
0.530
Alright well I'm going to grab QA of:- adding a plugin- remo…
3
content:
  content:
    p:
      score: 0
      elementCount: 8
    ul:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 6
    ol:
      score: 0
      elementCount: 1
  result: 3
regex:
  wordCount: 63
  wordValue: 0
  result: 0
0.730
Actions are failing due to env vars which I cannot edit I'm afra…
0
content:
  content:
    p:
      score: 0
      elementCount: 2
  result: 0
regex:
  wordCount: 81
  wordValue: 0
  result: 0
0.630
LFFGGGSome points for setup:1. Your DB auth should be tied t…
2.5
content:
  content:
    p:
      score: 0
      elementCount: 8
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 5
    ul:
      score: 0
      elementCount: 1
  result: 2.5
regex:
  wordCount: 190
  wordValue: 0
  result: 0
0.730
1. Modify an installed plugin2. Remove an installed plugin3.…
1.5
content:
  content:
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 3
    p:
      score: 0
      elementCount: 5
  result: 1.5
regex:
  wordCount: 65
  wordValue: 0
  result: 0
0.530
I'm going to move on and complete other tasks while reviews are …
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 14
  wordValue: 0
  result: 0
0.230
So the only changes requested I can implement is to ensure the a…
0
content:
  content:
    p:
      score: 0
      elementCount: 2
  result: 0
regex:
  wordCount: 31
  wordValue: 0
  result: 0
0.530
Yeah that's right, using the methods but the approach has change…
16.5
content:
  content:
    p:
      score: 0
      elementCount: 10
    a:
      score: 5
      elementCount: 1
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 3
    img:
      score: 5
      elementCount: 2
  result: 16.5
regex:
  wordCount: 130
  wordValue: 0
  result: 0
0.730
Np. It would be better if I could QA with the production app det…
5
content:
  content:
    p:
      score: 0
      elementCount: 4
    img:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 128
  wordValue: 0
  result: 0
0.330
Empty strings CI is not failing because of empty strings in the …
0
content:
  content:
    p:
      score: 0
      elementCount: 2
  result: 0
regex:
  wordCount: 57
  wordValue: 0
  result: 0
0.630
It was created under my account and then installed to my org yea…
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 33
  wordValue: 0
  result: 0
0.430
I just created a new app belonging to my org and everything work…
6.5
content:
  content:
    p:
      score: 0
      elementCount: 5
    ol:
      score: 0
      elementCount: 2
    li:
      score: 0.5
      elementCount: 3
    img:
      score: 5
      elementCount: 1
  result: 6.5
regex:
  wordCount: 80
  wordValue: 0
  result: 0
0.630
That was the intention yeahYou can't check installs using an O…
1.5
content:
  content:
    p:
      score: 0
      elementCount: 6
    hr:
      score: 0
      elementCount: 1
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 3
  result: 1.5
regex:
  wordCount: 128
  wordValue: 0
  result: 0
0.730
Sorry no it didn't, I assumed that would be part of the kernel s…
0
content:
  content:
    p:
      score: 0
      elementCount: 2
  result: 0
regex:
  wordCount: 38
  wordValue: 0
  result: 0
0.530
I'm not so sure. I'm using [YAML](https://github.com/eemeli/yaml…
5
content:
  content:
    p:
      score: 0
      elementCount: 3
    a:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 84
  wordValue: 0
  result: 0
0.330
I did this here [aba5a19](https://github.com/ubiquity-os/ubiquit…
5
content:
  content:
    p:
      score: 0
      elementCount: 1
    a:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 21
  wordValue: 0
  result: 0
0.230
My bad I didn't notice that but also weird that it failed due to…
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 16
  wordValue: 0
  result: 0
0.330
I cannot merge, so whenever you are ready is good.
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 10
  wordValue: 0
  result: 0
0.230

 [ 119.931 WXDAI ] 

@0x4007
Contributions Overview
ViewContributionCountReward
IssueSpecification1117.54
IssueComment10
ReviewComment32.391
Conversation Incentives
CommentFormattingRelevancePriorityReward
1. We need to be able to select which `ubiquibot-config.yml&…
13.06
content:
  content:
    ol:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 2
    p:
      score: 0
      elementCount: 5
    a:
      score: 5
      elementCount: 1
    h3:
      score: 1
      elementCount: 1
  result: 7
regex:
  wordCount: 125
  wordValue: 0.1
  result: 6.06
13117.54
@gentlementlegen can you fix the double post?
0.52
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 7
  wordValue: 0.1
  result: 0.52
030
cspell has weird bugs, I'm assuming there is no typo and its a f…
1.06
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 16
  wordValue: 0.1
  result: 1.06
0.230.636
This should be a secret. Like the ANON KEY
0.65
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 9
  wordValue: 0.1
  result: 0.65
0.931.755
```suggestionsha: sha,```
0
content:
  content: {}
  result: 0
regex:
  wordCount: 0
  wordValue: 0.1
  result: 0
0.530

 [ 34.152 WXDAI ] 

@gentlementlegen
Contributions Overview
ViewContributionCountReward
IssueComment32.727
ReviewComment1731.425
Conversation Incentives
CommentFormattingRelevancePriorityReward
@0x4007 Spec at https://github.com/ubiquity-os-marketplace/comma…
0.94
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 14
  wordValue: 0.1
  result: 0.94
0.531.41
Funny, I had made a tool for my personal usage a while ago, I di…
1.22
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 19
  wordValue: 0.1
  result: 1.22
0.230.732
@Keyrxng It is a very different approach! At first it was just f…
1.95
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 33
  wordValue: 0.1
  result: 1.95
0.130.585
Why is `issu` needed?
0.25
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 3
  wordValue: 0.1
  result: 0.25
0.830.6
Wouldn't it be just `SUPABASE_KEY`
0.39
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 5
  wordValue: 0.1
  result: 0.39
0.730.819
Ideally I think we should use the latest version of eslint that …
1.65
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 27
  wordValue: 0.1
  result: 1.65
0.934.455
@Keyrxng maybe you can add yourself 😄
0.46
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 6
  wordValue: 0.1
  result: 0.46
0.230.276
```suggestion```
0
content:
  content: {}
  result: 0
regex:
  wordCount: 0
  wordValue: 0.1
  result: 0
0.630
I think we just updated this template
0.52
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 7
  wordValue: 0.1
  result: 0.52
0.430.624
Would be better to run this on pull request events.
0.71
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 10
  wordValue: 0.1
  result: 0.71
0.831.704
This script will always fail, I made a correction to it: https:/…
1.22
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 19
  wordValue: 0.1
  result: 1.22
0.933.294
Please update the manifest accordingly.
0.39
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 5
  wordValue: 0.1
  result: 0.39
0.730.819
If that is meant to be an anon key then this is fine.
0.88
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 13
  wordValue: 0.1
  result: 0.88
0.330.792
Ah I see. Maybe you can fix the typo instead?
0.71
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 10
  wordValue: 0.1
  result: 0.71
0.731.491
Wouldn't the error test be more relevant?
0.59
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 8
  wordValue: 0.1
  result: 0.59
0.631.062
Maybe we could get these from the SDK?
0.59
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 8
  wordValue: 0.1
  result: 0.59
0.831.416
Might want to rename this to element instead of ele?
0.71
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 10
  wordValue: 0.1
  result: 0.71
0.531.065
@Keyrxng Yes it is, it means the module is disabled.
0.71
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 10
  wordValue: 0.1
  result: 0.71
0.330.639
Tested, works, that's cool.Some details that would be worth a …
4.61
content:
  content:
    p:
      score: 0
      elementCount: 4
    ul:
      score: 0
      elementCount: 2
    li:
      score: 0.5
      elementCount: 2
  result: 1
regex:
  wordCount: 68
  wordValue: 0.1
  result: 3.61
0.7310.581
If that's not causing any issue then don't sweat on it, I just w…
1.49
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 24
  wordValue: 0.1
  result: 1.49
0.431.788

 [ 185.784 WXDAI ] 

@rndquu
Contributions Overview
ViewContributionCountReward
ReviewComment6185.784
Conversation Incentives
CommentFormattingRelevancePriorityReward
1. Getting the `authentication.ts:127 GET https://api.github…
46.01
content:
  content:
    ol:
      score: 0
      elementCount: 2
    li:
      score: 0.5
      elementCount: 2
    p:
      score: 0
      elementCount: 7
    a:
      score: 5
      elementCount: 8
  result: 41
regex:
  wordCount: 100
  wordValue: 0.1
  result: 5.01
0.93136.527
@Keyrxng https://github.com/ubiquity-os/ubiquity-os-plugin-insta…
1.22
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 19
  wordValue: 0.1
  result: 1.22
0.230.732
@Keyrxng If partner doesn't have the `UbiquityOS` app …
3.15
content:
  content:
    p:
      score: 0
      elementCount: 4
  result: 0
regex:
  wordCount: 58
  wordValue: 0.1
  result: 3.15
0.736.615
@Keyrxng Trying to add the https://github.com/ubiquity-os-market…
7.97
content:
  content:
    p:
      score: 0
      elementCount: 3
    a:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 54
  wordValue: 0.1
  result: 2.97
0.8322.128
Awesome, works fine.@Keyrxng Could you update the readme file …
7.05
content:
  content:
    p:
      score: 0
      elementCount: 3
    a:
      score: 5
      elementCount: 1
  result: 5
regex:
  wordCount: 35
  wordValue: 0.1
  result: 2.05
0.6318.69
Could you fix the failing knip ci?
0.52
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 7
  wordValue: 0.1
  result: 0.52
0.731.092
gentlementlegen commented 1 day ago

@0x4007 that's one hell of a spec