Subsequent Events (Details) - Subsequent events - BCH $ in Thousands	Feb. 21, 2017 USD ($) installment
Subsequent Events
Fair value of consideration transferred	$ 100,000
Purchase price paid in cash	85,700
Promissory note payable	$ 14,300
Number of installments payable for promissory note \| installment	2
Promissory note payable in each installment	$ 7,150

Amount of liabilities incurred by the acquirer as part of consideration transferred in a business combination.

No definition available.

Name:	fix_BusinessCombinationConsiderationTransferredLiabilitiesIncurredPayableInEachInstallment
Namespace Prefix:	fix_
Data Type:	xbrli:monetaryItemType
Balance Type:	credit
Period Type:	duration

- Definition

Represents the number of installments of promissory note payable.

+ References

No definition available.

+ Details

Name:	fix_NumberOfInstallmentsPromissoryNotePayable
Namespace Prefix:	fix_
Data Type:	xbrli:integerItemType
Balance Type:	na
Period Type:	duration

- Definition

Amount of consideration transferred, consisting of acquisition-date fair value of assets transferred by the acquirer, liabilities incurred by the acquirer, and equity interest issued by the acquirer.

+ References

Reference 1: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 30
-Paragraph 7
-URI http://asc.fasb.org/extlink&oid=6911251&loc=d3e6578-128477

Reference 2: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 50
-Paragraph 1
-Subparagraph (b)
-URI http://asc.fasb.org/extlink&oid=35744584&loc=d3e6927-128479

Reference 3: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 30
-Paragraph 8
-URI http://asc.fasb.org/extlink&oid=6911251&loc=d3e6613-128477

+ Details

Name:	us-gaap_BusinessCombinationConsiderationTransferred1
Namespace Prefix:	us-gaap_
Data Type:	xbrli:monetaryItemType
Balance Type:	credit
Period Type:	duration

- Definition

Amount of liabilities incurred by the acquirer as part of consideration transferred in a business combination.

+ References

Reference 1: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 25
-Paragraph 5
-URI http://asc.fasb.org/extlink&oid=6911189&loc=d3e6405-128476

Reference 2: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 30
-Paragraph 7
-URI http://asc.fasb.org/extlink&oid=6911251&loc=d3e6578-128477

Reference 3: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 805
-SubTopic 30
-Section 30
-Paragraph 8
-URI http://asc.fasb.org/extlink&oid=6911251&loc=d3e6613-128477

+ Details

Name:	us-gaap_BusinessCombinationConsiderationTransferredLiabilitiesIncurred
Namespace Prefix:	us-gaap_
Data Type:	xbrli:monetaryItemType
Balance Type:	credit
Period Type:	duration

- Definition

The cash outflow associated with the acquisition of business during the period. The cash portion only of the acquisition price.

+ References

Reference 1: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Glossary Investing Activities
-URI http://asc.fasb.org/extlink&oid=6516133

Reference 2: http://www.xbrl.org/2003/role/presentationRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 230
-SubTopic 10
-Section 45
-Paragraph 13
-Subparagraph (b)
-URI http://asc.fasb.org/extlink&oid=66902209&loc=d3e3213-108585

+ Details

Name:	us-gaap_PaymentsToAcquireBusinessesGross
Namespace Prefix:	us-gaap_
Data Type:	xbrli:monetaryItemType
Balance Type:	credit
Period Type:	duration

- Definition

Detail information of subsequent event by type. User is expected to use existing line items from elsewhere in the taxonomy as the primary line items for this disclosure, which is further associated with dimension and member elements pertaining to a subsequent event.

+ References

No definition available.

+ Details

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

- Details

Name:	us-gaap_SubsequentEventTypeAxis=us-gaap_SubsequentEventMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type:

- Details

Name:	us-gaap_BusinessAcquisitionAxis=fix_BchHoldingsIncMember
Namespace Prefix:
Data Type:	na
Balance Type:
Period Type: