[clementbiron]

Validating modified declarations fetched every source document of a modified terms type, twice each, even when a single entry of its combine changed. Adding 4 documents to the Facebook "Advertising Content Policy" combine refetched its 46 documents: the validation of this pull request took 18 minutes on user-rights, and 64 minutes for the one adding Instagram.

ota validate declarations --modified now compares the declaration that each source document effectively gets on the default branch and on the branch, once the terms properties it inherits are applied as when loading the service, and fetches only the documents whose declaration changed. Documents are indexed by location, so reordering the combine fetches nothing and changing a location is treated as a new document. Terms added in the branch, new services and services whose filters changed keep no restriction, as any of their documents may be impacted.

The checks that do not fetch, such as the detection of duplicate source documents, still cover the whole combine, as an unchanged document can conflict with an added one. The number of skipped documents is logged to make the saving visible in the CI logs:

Terms of Service
  (3 of 4 source documents skipped as unchanged)
  ✔ does not declare the same source document more than once
  https://domain.example/tos/payments
    ✔ fetchable URL