Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/newtask #31

Open
wants to merge 7 commits into
base: development
Choose a base branch
from

Conversation

Keyrxng
Copy link
Member

@Keyrxng Keyrxng commented Nov 4, 2024

Resolves #32

this PR introduces the /newtask command which is used by bot admins to create new tasks in the correct repository.

Usage: /newtask ubiquity-os-marketplace/ubiquity-os-kernel-telegram

  • The admin must reply to the message which they want to turn into a task spec
  • They must include the repo.full_name after the command in order to open it in the correct repo
  • Only admins are able to use this new command
  • New tasks are currently opened under my alt account because the org secret APP_PRIVATE_KEY issue, no one has replied to me about it so I'm assuming it's still unresolved. It needs fixed ASAP.

QA: ubq-testing#9

Task creation comment:

The plugin DM interface needs to be standardized so that all plugins can easily use it to enable notifications for any reason.

Not just that but the logger can have a plugin built to enable it to be able to DM errors too. That feels like covering both the good and bad outcomes regarding notifications

image

@Keyrxng Keyrxng requested a review from 0x4007 November 4, 2024 02:24
@Keyrxng
Copy link
Member Author

Keyrxng commented Nov 4, 2024

  1. Using GPT to structure a proper spec.
  2. Using Directives and Constraints to guide, please offer suggestions on improvements.
  3. With embeddings I expect the specs to be more insightful and less generic.

Broke this into it's own PR and task via a task list, I think that's the right thing to do here. Once I get the embeddings data I can make moves on the other.

@Keyrxng
Copy link
Member Author

Keyrxng commented Nov 6, 2024

I saw the thumbs up on the task to my comment so I'll take that as a go-ahead and I'll merge this today tomorrow.

I fell into a rabbit whole with the logger error debugging and stacks in gh comments trying various solutions, found a few will PR that and close this tmrw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement /newtask
1 participant