As part of this change: overmindtech/sdp#12 there have been a number of changes to the protocol, namely:

• linkedItemSubject has been removed
• Errors are now included as part of a response, and the errorSubject has been removed

This repo needs to be refactored to work with these new changes

We now have very nice query status updates,, we should be tracking them here:

This will help with: https://github.com/overmindtech/frontend/issues/77

As per this issue, I need to add the itemRequest itself to the metadata. Once this is done in SDP, check the utilities here and see what needs updating

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Awaiting Schedule

These updates are awaiting their schedule. Click on a checkbox to get an update now.

• chore(deps): update all non-major dependencies (pnpm, vite)

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• chore(deps): update dependency @bufbuild/protobuf to v2

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

SDP v0.18 adds a load of new types and API endpoints. We need to work out what changes need to be made before this can be used:

• Get the tests passing
• See what needs to be done for the frontend to start consuming this
• Release

The API that is generate by the current generator is awful. We have considered another one in #12. But it doesn't support structs which is a dealbreaker. Maybe this one is a better option: https://buf.build/blog/protobuf-es-the-protocol-buffers-typescript-javascript-runtime-we-all-deserve

As part of overmindtech/sdp#12 subjects have been renamed to singular. This means that in this repo the following needs to be refactored:

• requests.all changed to request.all
• requests.contexts.{context} changed to request.context.{context}

The code generated by protoc-gen-ts is much nicer. However as far as I can tell it doesn't play nicely with well-known types such as google.protobuf.Struct which is a dealbreaker for me. See this conversation: thesayyn/protoc-gen-ts#129

If this is resolved though it'll definitely be better to change

I'm having issues with compiling the js code at the moment:

protoc-gen-js: program not found or is not executable
Please specify a program using absolute path or make sure the program is available in your PATH system variable

This is due to a bug in the latest release of protobuf: protocolbuffers/protobuf-javascript#127

Once this is fixed I need to:

• Generate the new TS files
• Create a method called Done() that tells if a request is fully complete, including post-processing

When calling Util.GetUniqueAttributeValue() on an item that has a non-string UAV, we get strange output. For example when called against a process for which the UAV is in integer 1, we get: ",1"

Need to add some spec testing so that I can test for this sort of thing easily as we don't currently have that

It will be good for the UI to be able to intelligently autocomplete as per this ticket: overmindtech/discovery#56

In order for this to be easy to implement in the front-end, we should create a wrapper that returns autocomplete suggestions for a given string.

I tried this initially and even though it passed all the tests I was getting an error at runtime:

"X is not a Constructor"

When trying to use the constructors of any of the objects from the different file, even though they were being imported and re-exported in the index.ts file. I looked at a few different ways to try and get it to export them correctly and the only way that I could get it to work was to have everything in the one file, which sucks