Assume that scope IDs are opaque key objects

[kdmccormick]

Assume that scope IDs are opaque key objects

[kdmccormick]

@bradenmacdonald , Here's the current ScopeIds definition:

class ScopeIds(namedtuple('ScopeIds', 'user_id block_type def_id usage_id')):
    ....

I'm thinking we could change it to something like:

class ScopeIds(NamedTuple):
    user_id: int  # or just 'object'?
    block_type:  str
    def_id: DefinitionKey
    usage_id: UsageKey

do you agree?

[bradenmacdonald]

This is great, @kdmccormick

Just need to investigate the type of user_id a bit more. I know it can be None sometimes when an XBlock is not "bound" to a particular user. And at least in the "new runtime", an anonymous user gets a string ID. So I think it should be typed as int|str|None.

Example: https://github.com/openedx/edx-platform/blob/2cd1410d9951f0b5cd1639f25dd34e191d4d75f3/openedx/core/djangoapps/xblock/runtime/runtime.py#L110-L117

[kdmccormick]

I'm currently doing this in tandem with #707 , implemented by #713.

My thinking is that we'll make the change, bump the major version of XBlock, and pin xblock-sdk to an older version of XBlock.

Following up, we should either deprecate xblock-sdk, or a find a maintainer for it who is willing to remove the XBlock version pin.