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

[Needed for Discord bot] Segment Interview and Spec Generation #62

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Feb 28, 2024

This turns the Spec generation into the same style as the other blocks and extracts the interview stage and modifies it a bit to support a bit more of a rest style api. The codex client is now responsible for stepping through the interview phases and answering questions that come with each phase before ultimately calling generate spec

Also requires a cloud service id and interview id for Identifiers now

may want to consider seeding our users into CODEX and Cloud Services with compatible Ids and correct discords and such for core devs of the services

Copy link

linear bot commented Feb 28, 2024

AGPT-400

@ntindle ntindle marked this pull request as ready for review February 29, 2024 04:47
@ntindle
Copy link
Member Author

ntindle commented Feb 29, 2024

Todo:

  • swap port back or change it permanently. Collides with proxy and is a pain to run whole stack with it on 8000
  • Test from /docs that it works through and through
  • Make sure the spec can still happen and uses the correct details
  • Update the various test systems
  • Check to make sure search works
  • Check to make sure that unanswered asks get answered automatically

@ntindle ntindle changed the title Update to interview and backgorund schedule specs Segment Interview and Spec Generation Feb 29, 2024
@ntindle ntindle requested a review from majdyz February 29, 2024 06:00
@ntindle ntindle marked this pull request as draft February 29, 2024 06:00
@ntindle ntindle marked this pull request as ready for review February 29, 2024 20:18
@ntindle ntindle changed the title Segment Interview and Spec Generation [Needed for Discord bot] Segment Interview and Spec Generation Mar 1, 2024
Copy link
Contributor

@majdyz majdyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I hope we can follow-up on adding one test on test_api.py to prevent from us breaking this code :)

migrations/20240229030242_add_interview/migration.sql Outdated Show resolved Hide resolved
schema.prisma Outdated Show resolved Hide resolved
codex/requirements/routes.py Outdated Show resolved Hide resolved
codex/requirements/routes.py Show resolved Hide resolved
@ntindle ntindle merged commit 10da924 into main Mar 1, 2024
1 check passed
@ntindle ntindle deleted the ntindle/agpt-400-update-post-spec-to-schedule-a-background-task-for branch March 1, 2024 05:14
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.

2 participants