DictDescriptorStore (server/app/model/provider.py) has no commit() method, so it uses the SDK's default one, which just raises NotImplementedError(). The registry calls commit() on every add/update, so POST/PUT to /shell-descriptors returns 500 in the default config.
Proposed fix: add a no-op commit() to DictDescriptorStore, like the SDK's DictObjectStore already has.
Root cause is that the commit() method in the abstract base class AbstractObjectStore is not flagged as an abstract method.
We should consider adding the correct decorator.
DictDescriptorStore(server/app/model/provider.py) has nocommit()method, so it uses the SDK's default one, which just raisesNotImplementedError(). The registry callscommit()on every add/update, soPOST/PUTto/shell-descriptorsreturns500in the default config.Proposed fix: add a no-op
commit()toDictDescriptorStore, like the SDK'sDictObjectStorealready has.Root cause is that the
commit()method in the abstract base classAbstractObjectStoreis not flagged as an abstract method.We should consider adding the correct decorator.