Use globally unique filekeys for referencing other yaml files

[kevinbackhouse]

The main change is in yaml_parser.py where I've removed the optional recurse and dir_namespace parameters. It now always recurses and always creates a namespaced name. Also, the name is no longer defined by the filename, but instead by the "filekey" that is defined in the yaml, like this:

seclab-taskflow-agent:
  version: 1
  filetype: personality
  filekey: GitHubSecurityLab/seclab-taskflow-agent/personalities/examples/fruit_expert

This has the benefit that a yaml file can moved to a different directory without breaking any cross-references. It also means that we can gather yaml files from a variety of sources, dump them all in one directory tree, and easily start using them.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[m-y-mo]

Could you also change the README to reflect this change? Thanks. (I think from here onwards: https://github.com/GitHubSecurityLab/seclab-taskflow-agent?tab=readme-ov-file#personalities)

[kevinbackhouse]

This is now updated to use "filekeys" for cross-referencing.

[m-y-mo]

Can we drop the .yaml extension in filekeys because everything is yaml so it doesn't really add anything other than more typing? Also the echo toolbox doesn't seem to have the .yaml extension in the filekeys. Also can we use dots instead of slash for namespacing, e.g. personalities/action_expert.yaml -> personalities.action_expert so it is clear that we are using filekeys and not filepath for referencing?

[kevinbackhouse]

smoke test