Acquisitions - Acquired Intangible Assets (Details) - Walker $ in Thousands	Apr. 01, 2019 USD ($)
Acquired Finite-Lived Intangible Assets [Line Items]
Estimated Fair Value	$ 90,200
Backlog
Acquired Finite-Lived Intangible Assets [Line Items]
Estimated Useful Life	2 years
Estimated Fair Value	$ 4,600
Trade Names
Acquired Finite-Lived Intangible Assets [Line Items]
Estimated Useful Life	25 years
Estimated Fair Value	$ 32,600
Customer Relationships
Acquired Finite-Lived Intangible Assets [Line Items]
Estimated Useful Life	10 years
Estimated Fair Value	$ 53,000

Line items represent financial concepts included in a table. These concepts are used to disclose reportable information associated with domain members defined in one or many axes to the table.

No definition available.

+ Details

Name:	us-gaap_AcquiredFiniteLivedIntangibleAssetsLineItems
Namespace Prefix:	us-gaap_
Data Type:	xbrli:stringItemType
Balance Type:	na
Period Type:	duration

- Definition

Weighted average amortization period of finite-lived intangible assets acquired either individually or as part of a group of assets, in 'PnYnMnDTnHnMnS' format, for example, 'P1Y5M13D' represents the reported fact of one year, five months, and thirteen days.

+ References

Reference 1: http://fasb.org/us-gaap/role/ref/legacyRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 350
-SubTopic 30
-Section 50
-Paragraph 1
-Subparagraph (a)(3)
-URI http://asc.fasb.org/extlink&oid=66006027&loc=d3e16265-109275

+ Details

Name:	us-gaap_AcquiredFiniteLivedIntangibleAssetsWeightedAverageUsefulLife
Namespace Prefix:	us-gaap_
Data Type:	xbrli:durationItemType
Balance Type:	na
Period Type:	duration

- Definition

The amount of identifiable intangible assets recognized as of the acquisition date.

+ References

Reference 1: http://fasb.org/us-gaap/role/ref/legacyRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 10
-Section 55
-Paragraph 37
-URI http://asc.fasb.org/extlink&oid=116868840&loc=d3e2207-128464

Reference 2: http://fasb.org/us-gaap/role/ref/legacyRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 20
-Section 50
-Paragraph 1
-Subparagraph (c)
-URI http://asc.fasb.org/extlink&oid=116869197&loc=d3e4845-128472

+ Details

Name:	us-gaap_BusinessCombinationRecognizedIdentifiableAssetsAcquiredAndLiabilitiesAssumedIntangibles
Namespace Prefix:	us-gaap_
Data Type:	xbrli:monetaryItemType
Balance Type:	debit
Period Type:	instant

- Details

Name:	us-gaap_BusinessAcquisitionAxis=fix_WalkerTxHoldingsInc.AndItsWhollyOwnedSubsidiariesMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type:

- Details

Name:	us-gaap_FiniteLivedIntangibleAssetsByMajorClassAxis=us-gaap_OrderOrProductionBacklogMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type:

- Details

Name:	us-gaap_FiniteLivedIntangibleAssetsByMajorClassAxis=us-gaap_TradeNamesMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type:

- Details

Name:	us-gaap_FiniteLivedIntangibleAssetsByMajorClassAxis=us-gaap_CustomerRelationshipsMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type: