Skip to content

Add stubs for resolvers - #1557

Merged
ia0 merged 5 commits into
google:mainfrom
ia0:resolvers
Oct 23, 2026
Merged

ia0 merged 5 commits into
google:mainfrom
ia0:resolvers

Conversation

@ia0

@ia0 ia0 commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

CC @corkamig

The flow is:

  • Identify with custom rules (and exit if there's a match): the rationale is that it's for hot-fixes
  • Identify with resolvers (and exit if there's a match)
  • Identify with builtin rules (and exit if there's a match): those will eventually disappear once resolvers cover them
  • Identify with model (but rules and resolvers may veto)

There is still one thing to be done, but it can be done later since it needs KB synchronization: the same way we have a column for "supported by the model" and "supported by the rules" for each content type, we also want a column for the resolvers. I think the simplest way to do it is to have a rust/gen/resolvers/content_types the same way we have rust/gen/rules/content_types except it would be manually edited instead of autogenerated.

@ia0
ia0 requested a review from reyammer October 22, 2026 10:24
@ia0
ia0 requested a review from invernizzi as a code owner October 22, 2026 10:24
@ia0
ia0 merged commit 248d727 into google:main Oct 23, 2026
30 checks passed
@ia0
ia0 deleted the resolvers branch October 23, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants