epam / miew / 29459414975

27%

main: 27%

push

github

Add nohetatm mode to autoBonding setting (#615)

Implement the 'nohetatm' mode that was deferred in the original
autoBonding setting commit. When selected, HETATM atoms are excluded
from auto-bonding in both directions (as a seed and as a candidate
neighbor), while the rest of the structure still bonds per the
format's default behavior.

- Add `AutoBond` constructor option `excludeHetatm` and make
  `_isAtomEligible()` an instance method that respects it; filter
  candidate neighbor atoms too, closing an existing asymmetry where
  only the seed atom was checked.
- Add `Parser#excludeHetatmFromAutoBonding()`, sharing mode resolution
  with `resolveAutoBonding()` via a new `_resolveAutoBondingMode()`.
- Wire the new option into `Complex#finalize()` and only into
  PDBParser, CIFParser, and MMTFParser, since those are the only
  parsers where the `het` flag is a genuine per-atom distinction read
  from the source file. GROParser, XYZParser, CMLParser, SDFParser,
  PubChemParser, and MOL2Parser hardcode `het` to a single constant
  for every atom, so wiring the exclusion there would either silently
  drop all auto-generated bonds (GRO, XYZ) or be dead code.
- Document the new mode in settings.js and Miew#load().
- Add tests for `excludeHetatmFromAutoBonding()` and a new
  `AutoBond.test.js` covering the exclusion behavior end-to-end via
  `Complex#finalize()`.

Co-authored-by: GitHub Copilot <copilot@github.com>

1495 of 5749 branches covered (26.0%)

Branch coverage included in aggregate %.

11 of 11 new or added lines in 3 files covered. (100.0%)

4337 of 15902 relevant lines covered (27.27%)

57938.0 hits per line