hash icon indicating copy to clipboard operation
hash copied to clipboard

H-2711, H-2712, H-2715: Add `/goals` and `/goals/new` pages

Open CiaranMn opened this issue 1 year ago β€’ 1 comments

🌟 What is the purpose of this PR?

Adds the following pages:

  1. /goals, which lists all active goal tasks (goals are research-driven flows)
  2. /goals/new, which allows creating a new goal task

For now, the goals task is a special flow definition. Later on the flow definition itself will be generated by AI.

Internal designs are here

Driveby:

  • H-2712: expose model plan/explanation for actions as part of progress logs (see second demo)
  • H-2715: In the browser plugin history log, show where the plugin has responded to a flow's request for a web page content

πŸ” What does this change?

  • ...

Pre-Merge Checklist πŸš€

🚒 Has this modified a publishable library?

This PR:

  • [x] does not modify any publishable blocks or libraries, or modifications do not need publishing

πŸ“œ Does this require a change to the docs?

The changes in this PR:

  • [x] are internal and do not require a docs change

πŸ•ΈοΈ Does this require a change to the Turbo Graph?

The changes in this PR:

  • [x] do not affect the execution graph
  • [x] affected the execution graph, and the turbo.json's have been updated to reflect this
  • [x] I am unsure / need advice

⚠️ Known issues

  • H-2722: The goals should have distinguishing names based on the research prompt somehow

πŸ›‘ What tests cover this?

  • None yet

❓ How to test this?

  1. Check out the new pages

πŸ“Ή Demo

New pages

https://github.com/hashintel/hash/assets/37743469/556529d3-a16e-4228-b751-0ffd574fb1c3

Progress log tooltips with model plan / explanation of actions

https://github.com/hashintel/hash/assets/37743469/90868ff1-ef9f-4d9b-924c-d38a8f7ca0ef

CiaranMn avatar May 19 '24 14:05 CiaranMn

Codecov Report

Attention: Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.

Project coverage is 21.15%. Comparing base (7ef056a) to head (e5c5f16). Report is 2050 commits behind head on main.

Files with missing lines Patch % Lines
...sh-ai-worker-ts/src/workflows/run-flow-workflow.ts 0.00% 9 Missing :warning:
...l/hash-backend-utils/flows/get-flow-run-details.ts 0.00% 5 Missing :warning:
...vities/flow-activities/research-entities-action.ts 0.00% 3 Missing :warning:
apps/hash-api/src/ai/infer-entities-websocket.ts 0.00% 2 Missing :warning:
...w-activities/infer-entities-from-content-action.ts 0.00% 1 Missing :warning:
...orphic-utils/src/flows/example-flow-definitions.ts 0.00% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4485      +/-   ##
==========================================
- Coverage   21.44%   21.15%   -0.30%     
==========================================
  Files         453      456       +3     
  Lines       15007    15215     +208     
  Branches     2221     2273      +52     
==========================================
  Hits         3218     3218              
- Misses      11748    11956     +208     
  Partials       41       41              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar May 19 '24 14:05 codecov[bot]

@benwerner01 I fixed the bug in the popup you identified in https://github.com/hashintel/hash/pull/4485/commits/e5c5f16e256299417b7b7ce219f94ec99fc3b3f9, as well as another one where the user was being overfetched, causing settings changes to be overwritten sometimes

CiaranMn avatar May 23 '24 10:05 CiaranMn