apollo: Remove nested adaptor functions in AI assistant

[hunterachieng]

Short Description

The AI assistant currently gives code snippets for job writing that have nested adaptor functions which goes away from the openFn job writing architectural requirement.

Fixes #97

Implementation Details

A more detailed breakdown of the changes, including motivations (if not provided in the issue).

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to know!):

• Code generation (copilot but not intellisense)
• Learning or fact checking
• Strategy / design
• Optimisation / refactoring
• Translation / spellchecking / doc gen
• Other
• I have not used AI

You can read more details in our Responsible AI Policy

[josephjclark]

Can I ask you to rename both notebooks to something more meanigful? issue 97 is too broad.

Maybe like:

test_nested_state
test_related_questions

[josephjclark]

For your test, I would argue that we don;t need to check whether it answers the question. It's sufficient just to look for that nested state argument.

I'd also argue you can do this with regular python rather than using AI? But I'll leave that up to you

[josephjclark]

The success rate here on v1 is way too high!

I think that means you'll need to manually figure out questions which showcase the issue. Or maybe generate many more questions.

We can only solve the problem if we have a good dataset of questions which showcase the issue :)

[hunterachieng]

@josephjclark I have fixed the requested changes. For the v1 and v2 issue, they might look the same since I am using the same Job_guide in both. I had to import the existing one. Maybe if we edit is when we can notice a difference?