fix(filter) : remove imports of filtered types

[dselman]

Closes #682

Updates the BaseModelManager.filter method to update all the imports in the model manager based on the types that were removed by calls to ModelFiler.filter.

Changes

• ModelFile.filter method updated to return an array of the types that were removed by the filter predicate
• BaseModelManager.filter method updated to remove imports of removed types
• Unit test

Flags

• The change to ModelFile.filter to take removedDeclarations as an extra argument is technically a breaking change however if we assume that the only caller of this function is BaseModelManager then we could argue that this is a pure bug fix.
• I'm uncertain about this behaviour: ModelFiles with no declarations after filtering will be removed. — I suspect this could break wildcard imports when Concerto is used with strict:false. I'm choosing to ignore this given folks should be using strict:true and the likelihood that they are not and also using filter is small.

Screenshots or Video

Related Issues

• Issue #
• Pull Request #

Author Checklist

• Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
• Vital features and changes captured in unit and/or integration tests
• Commits messages follow AP format
• Extend the documentation, if necessary
• Merging to main from fork:branchname